<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Feb 6, 2013, at 4:59 PM, Alec Flett &lt;<a href="mailto:alecflett@chromium.org">alecflett@chromium.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div class="im" style="font-family:arial,sans-serif;font-size:14px">On Wed, Feb 6, 2013 at 4:48 PM, Maciej Stachowiak&nbsp;<span dir="ltr">&lt;<a href="mailto:mjs@apple.com" target="_blank" class="cremed">mjs@apple.com</a>&gt;</span>&nbsp;wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br></div><div>I think we should continue to use uint8_t instead of char as the primary way to represent a raw byte in WebKit. First, it's good to distinguish raw data from C strings at the type system level, and second, the unpredictable signedness of char is actively bad for byte-oriented processing. Another library making a different choice doesn't overcome these reasons.</div>

<div><br></div></div></blockquote></div><div style="font-family:arial,sans-serif;font-size:14px">To be fair, there hasn't been a convention in WebKit at all. &nbsp;uint8_t was chosen for SerializedScriptValue roughly two months ago, with specific IndexedDB support in mind:&nbsp;<a href="https://bugs.webkit.org/show_bug.cgi?id=104354" target="_blank" class="cremed">https://bugs.webkit.org/show_bug.cgi?id=104354</a>&nbsp;- this usage is not widespread, and in fact the only consumer of this type is IndexedDB.</div></div></blockquote><div><br></div><div>I don't know where you get this idea from -- SerializedScriptValue has always used uint8 buffers. &nbsp;That IndexedDB used Strings and that _it_ changed to uint8 buffers is entirely irrelevant here.</div></div><br><div>--Oliver</div></body></html>