<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 15, 2014 at 3:23 AM, Frédéric Riss <span dir="ltr"><<a href="mailto:friss@apple.com" target="_blank">friss@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>This is obviously true, but I don’t see it with my system clang version, and I didn’t see it on any on the bots I looked at. Maybe it will show up in later bootstrap builds that will use trunk clang.</div><div><br></div><div>As I need pre-commit review to get the fix in, do you want me to revert the change now?</div></blockquote><div><br></div><div>Whaaa? No no, you don't need pre-commit review to fix a problem pointed out by a warning. =D Please just commit the fix if you have it. If not, I'll commit it shortly so my builds are tidied up.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div> I’ll submit the code removing the inheritance later today, but I can’t presume of the review time.</div></blockquote></div><br><br></div></div>