<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="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="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888">Galina<br></font></span></div>
</blockquote></div><br></div>