<div dir="ltr">Thanks. Addressed in <span style="color:rgb(0,0,0)">r238675. </span></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, May 30, 2015 at 10:10 PM, Simon Atanasyan <span dir="ltr"><<a href="mailto:simon@atanasyan.com" target="_blank">simon@atanasyan.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<span class=""><br>
On Sun, May 31, 2015 at 6:57 AM, Rui Ueyama <<a href="mailto:ruiu@google.com">ruiu@google.com</a>> wrote:<br>
> @@ -172,7 +174,7 @@ ErrorOr<StringRef> SymbolTable::findDefa<br>
>        {"WinMain", "WinMainCRTStartup"},<br>
>        {"wWinMain", "wWinMainCRTStartup"},<br>
>    };<br>
> -  for (size_t I = 0; I < sizeof(Entries); ++I) {<br>
> +  for (size_t I = 0; I < array_lengthof(Entries); ++I) {<br>
>      if (!find(Entries[I][0]))<br>
>        continue;<br>
>      if (auto EC = addSymbol(new Undefined(Entries[I][1])))<br>
<br>
</span>Maybe the range loop based approach is even safer and shorter:<br>
[[<br>
for (const auto &Entry : Entries) {<br>
  if (!find(Entry[0]))<br>
    continue;<br>
  if (auto EC = addSymbol(new Undefined(Entry[1])))<br>
    return EC;<br>
  return StringRef(Entry[1]);<br>
}<br>
]]<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Simon Atanasyan<br>
</font></span></blockquote></div><br></div>