<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 26, 2015 at 11:14 AM, George Rimar <span dir="ltr"><<a href="mailto:grimar@accesssoftek.com" target="_blank">grimar@accesssoftek.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">>If it helps, I'll make it a post-commit review request: please change this to auto because of the subtleties here.<br>
><br>
>Was this caught by a buildbot with Clang's warning, or due to a test failing on a machine where size_t != unsigned?<br>
><br>
>- Dave<br>
<br>
<br>
</span>If failed due to build bot`s error where size_t != unsigned I guess. My msvs/32x config compiled fine that place.<br></blockquote><div><br></div><div>OK - I think we have a warning for this too, but maybe it hasn't been rolled out. Not sure. (cc'd Richard Trieu who implemented/worked on the warning, to see what the status is)<br><br>But yeah, especially if this actually introduced a bug (not just a compiler warning), I'd really prefer auto here. These sort of mistakes are pretty easy.</div></div><br></div></div>