<div class="gmail_quote">On Thu, Dec 8, 2011 at 12:45 PM, Delesley Hutchins <span dir="ltr"><<a href="mailto:delesley@google.com">delesley@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 id=":5pa">I don't like the second warning, but it's not necessarily all that<br>
easy to get rid of it.  As with many warnings, once the compiler is<br>
confused, you may get several warnings due to one mistake.  It's hard<br>
to avoid that in all cases.</div></blockquote></div><br><div>But that's not really the case here. The compiler isn't confused.</div><div><br></div><div>It seems like when you are warning about an invalid locking pattern, you should mark the aspects of that pattern already warned about as invalid to suppress subsequent warnings on them.</div>
<div><br></div><div>For example, when you see a lock acquired a second time, warn about it, but then suppress both the acquisition and the subsequent (paired) release from the lockset so that you don't get further warnings.</div>