<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Hi all,<br class=""><br class="">I recently started updating the WTF::Vector API to use unsigned types instead of size_t [1][2], because:<br class="">- WTF::Vector is already using unsigned type for size/capacity internally to save memory on 64-bit, causing a mismatch between the API and the internal representation [3]<br class="">- Some reviewers have asked me to use unsigned for loop counters iterating over vectors (which looks unsafe as the Vector API, e.g. size(), returns a size_t).<br class="">- I heard from Joseph that this type mismatch is forcing us (and other projects using WTF) to disable some build time warnings<br class="">- The few people I talked to before making that change said we should do it<br class=""><br class="">However, Alexey recently raised concerns about this change. it doesn't "strike him as the right direction. 4Gb is not much, and we should have more of WebKit work with the right data types, not less.”.<br class="">I did not initially realize that this change was controversial, but now that it seems it is, I thought I would raise the question on webkit-dev to see what people think about this.<br class=""><br class="">Kr,<br class=""><div apple-content-edited="true" class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">--</div><div class="">Chris Dumez - Apple Inc.</div><div class="">Cupertino, CA</div><div class=""><br class=""></div></div><br class="Apple-interchange-newline">[1] <a href="http://trac.webkit.org/changeset/176275" class="">http://trac.webkit.org/changeset/176275</a><br class="">[2] <a href="http://trac.webkit.org/changeset/176293" class="">http://trac.webkit.org/changeset/176293</a><br class="">[3] <a href="http://trac.webkit.org/changeset/148891" class="">http://trac.webkit.org/changeset/148891</a>
</div>
<br class=""></body></html>