<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div>
<blockquote type="cite" class="">
<div class="">
<div dir="ltr" class="">
<div class="gmail_quote">
<div class=""><br class="">
</div>
<div class="">Another consideration regarding <span style="font-family:monospace" class=""><a class="gmail_plusreply" id="plusReplyChip-1">@apiNote</a></span> is how likely the problem really is. For example, several warnings come from code like this:</div>
<div class=""><br class="">
</div>
<div style="margin-left:40px" class=""><span style="font-family:monospace" class="">private PropertyChangeSupport support = new PropertyChangeSupport(this);</span></div>
<div class=""><br class="">
</div>
<div class="">Yes, it's possible in theory that <span style="font-family:monospace" class="">
PropertyChangeSupport</span> could someday be modified to invoke some method of 'this', but in reality we know all it does is stash 'this' in a field and return and that's not likely to change anytime soon. Not much to be gained from including an
<span style="font-family:monospace" class="">@apiNote</span> warning in those cases.<br class="">
</div>
</div>
</div>
</div>
</blockquote>
<div><br class="">
</div>
<div>Yeah, this is a mess. The set of conditions that one would have to check to prove this safe is large, and spans both the behavior of PCS and the client. If we are convinced that _specific uses_ of this are OK, then that’s a reasonable place to use @SW
without further note, but for things like the ArrayList constructors that call add, this is fundamentally dangerous and probably wants a note.</div>
<br class="">
<blockquote type="cite" class="">
<div dir="ltr" class="">
<div class="gmail_quote">
<div class="">FYI, I used <span class="" style="font-family: monospace;">@implNote</span> instead of
<span class="" style="font-family: monospace;">@apiNote</span> because this seemed like more of an implementation issue than an API issue (but please correct me if I'm wrong).</div>
</div>
</div>
</blockquote>
<div><br class="">
</div>
<div>It’s on the fence….</div>
<div><br class="">
</div>
<br class="">
<blockquote type="cite" class="">
<div dir="ltr" class="">
<div class="gmail_quote">
<div class=""><br class="">
</div>
<div class="">Also, since the last post I found and fixed a few bugs, which revealed more 'this' escapes. Here are the new numbers, which have gone up a bit:<br class="">
</div>
</div>
</div>
</blockquote>
<div><br class="">
</div>
<div>Are there interesting examples of the new form that we didn’t detect before, that would be educational to share?</div>
<br class="">
<br class="">
</div>
<br class="">
</body>
</html>