<div dir="ltr"><div style>At the moment, SerializedScriptValue uses Vector&lt;uint8_t&gt; (aka Vector&lt;unsigned char&gt;) for both it&#39;s API (createFromWireBytes, toWireBytes) as well as its internal representation. (for both v8 and jsc implementations)</div>

<div style><br></div><div style>The two largest consumers of this aspect of SerializedScriptValue seems to be IndexedDB and postMessage(). </div><div style><br></div><div style>I&#39;m jumping through some small hoops (i.e. reinterpret_cast and whatnot) in IndexedDB to convert between Vector&lt;uint8_t&gt; and Vector&lt;char&gt; and a &#39;const char*&#39; buffer in order to write out to LevelDB, who likes &#39;char&#39; as opposed to unsigned char. </div>

<div style><br></div><div style>postMessage() seems to be pretty agnostic to char vs unsigned char, since it uses SerializedScriptValue to both produce and consume the buffers it sends between windows.</div><div style><br>

</div><div style>Before I did a code cleanup and just fixed up all the implementations to use Vector&lt;char&gt; I wanted to see if anyone had any objections here, on both the V8 and the JSC sides. The ultimate compiled code is going to be identical, but I&#39;ll be able to avoid all sorts of reinterpret_cast&#39;s at various points in the code.</div>

<div style><br></div><div style>objections?</div><div style><br></div><div style>Alec</div></div>