<div dir="ltr">On Wed, Apr 3, 2013 at 10:05 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><p dir="ltr"><br>
On Apr 4, 2013 4:52 AM, "Dmitry Vyukov" <<a href="mailto:dvyukov@google.com" target="_blank">dvyukov@google.com</a>> wrote:<br>
><br>
><br>
><br>
> ================<br>
> Comment at: lib/sanitizer_common/sanitizer_common.h:114<br>
> @@ -112,1 +113,3 @@<br>
> +// Can be used to prevent mixing error reports from different sanitizers.<br>
> +extern StaticSpinMutex CommonSanitizerReportMutex;<br>
><br>
> ----------------<br>
> We usually don't name variables LikeThis, we usually name them like_this.<br>
> And quite verbose on my taste. But feel free to ignore.<br>
><br>
> ================<br>
> Comment at: lib/tsan/rtl/tsan_rtl.h:525<br>
> @@ -524,3 +524,2 @@<br>
><br>
> -  Mutex report_mtx;<br>
>    int nreported;<br>
> ----------------<br>
> Leave this one as well. It detects deadlocks.<br>
> And it must the outer mutex, because it measures contention as well.</p>
</div></div><p dir="ltr">Must what?</p>
<p style>Must be.</p></blockquote></div></div></div>