<div dir="ltr">Well the class holds only a pointer to a sys::Mutex, so in theory it can be used in any many possible ways.  I think this unit test is only intended to test the class itself, not clients of the class.</div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Jun 17, 2014 at 11:17 AM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@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 class="">On 17 June 2014 13:53, Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br>
> If you want a hint, everywhere is a recursive mutex except, as far as I can<br>
> tell, in the ValueMapTest in unittests, which uses a non recursive mutex.<br>
> Because recursive seems to be the most common, i made it the default<br>
> template argument type so that an existing user will not have to change his<br>
> syntax to compile with the new template arg.<br>
<br>
<br>
</div>That sounds like a bug in the test. Why is the test doing something<br>
different from the actual code using the class?<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>