<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 9, 2014 at 6:02 AM, Alp Toker <span dir="ltr"><<a href="mailto:alp@nuanti.com" target="_blank">alp@nuanti.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":10q" style="overflow:hidden">Hi Kostya,<br>
<br>
This doesn't look correct and fires warnings my my strict checks build.<br></div></blockquote><div><br></div><div>I'm pretty sure the change works correctly, and it was definitely what Kostya and I intended.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":10q" style="overflow:hidden">
<br>
Double-underscore prefixed identifiers are strictly reserved for the implementation.</div></blockquote></div><br>Yes, and LeakSanitizer is something provided by the implementation? (the toolchain's runtime libraries specifically) I don't see any problem with double underscore here, any more than with __builtin_... functions or any of the other sanitizer interfaces.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Can you clarify exactly what the error is and why? It's possible we could provide a macro of some kind in the sanitizer headers to bundle this functionality up if we need some special marking of this for checkers, but without more information its hard to hell what the actual problem is.</div>
</div>