<div dir="ltr"><div>From <span style="font-size:12.8px">Galinas' commit</span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">// TODO: Name.Offset.Offset here and in the all similar places below<br>// suggests a names refactoring. Maybe StringTableOffset.Value?</blockquote><div> See <span style="font-size:12.8px">rL305029</span></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 9, 2017 at 1:10 AM, Rui Ueyama <span dir="ltr"><<a href="mailto:ruiu@google.com" target="_blank">ruiu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Which names are you talking about?</div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jun 8, 2017 at 5:02 PM, Martell Malone <span dir="ltr"><<a href="mailto:martellmalone@gmail.com" target="_blank">martellmalone@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">LGTM<br>Removing the cast with a note about naming makes a lot of sense here.<br><br>Thanks Galina for finding a reasonable solution to fix the warnings.<br>Rui, I can follow up with a commit on the naming if you have a suggestion?<br><br>Best,<br>Martell</div><div class="m_-757804357451889531HOEnZb"><div class="m_-757804357451889531h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jun 9, 2017 at 12:38 AM, Galina Kistanova <span dir="ltr"><<a href="mailto:gkistanova@gmail.com" target="_blank">gkistanova@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div>I have committed the fix as r305029.<br><br></div>Naming there looks a bit odd, but this is for somebody else to address. :)<br><br></div>Thanks<span class="m_-757804357451889531m_5168672831555596104HOEnZb"><font color="#888888"><br><br></font></span></div><span class="m_-757804357451889531m_5168672831555596104HOEnZb"><font color="#888888">Galina<br></font></span></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>