<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 28, 2014 at 7:28 PM, JF Bastien <span dir="ltr"><<a href="mailto:jfb@google.com" target="_blank">jfb@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">I agree with Mark's assessment: his patch is better than the current state of things (silently saying a mutex was acquired when no such thing was done, causing non-deterministic crashes); implementing RWMutex for a few uses and only for a platform that doesn't have them is silly (when really NaCl should have them at some point, but that doesn't remove the need for this patch).<div>

<br></div><div>His patch makes it so that the next platform without RWMutex doesn't have to debug this random crash. Overall lgtm.</div></div></blockquote></div><br>:: eye roll ::</div><div class="gmail_extra"><br></div>
<div class="gmail_extra">In the time we've spent defending this patch, we could have implemented a real RWMutex. Fine, submit, I'll replace it later when I replace Mutex with std::mutex.</div></div>