<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body>
<a class="moz-txt-link-freetext" href="https://bugs.openjdk.org/browse/JDK-8344697">https://bugs.openjdk.org/browse/JDK-8344697</a> is closed as a dup.<br>
At this point I'd say A, although re-opening and renaming is
possible.<br>
<br>
-phil<br>
<br>
<br>
<div class="moz-cite-prefix">On 4/11/25 9:58 AM, Jeremy Wood wrote:<br>
</div>
<blockquote type="cite" cite="mid:em05939e17-3f66-47b3-a990-949a2afbca11@f434645e.com">
<style id="css_styles">blockquote.cite { margin-left: 5px; margin-right: 0px; padding-left: 10px; padding-right:0px; border-left: 1px solid #cccccc }blockquote.cite2 {margin-left: 5px; margin-right: 0px; padding-left: 10px; padding-right:0px; border-left: 1px solid #cccccc; margin-top: 3px; padding-top: 0px; }a img { border: 0px; }li[style='text-align: center;'], li[style='text-align: center; '], li[style='text-align: right;'], li[style='text-align: right; '] { list-style-position: inside;}body { font-family: Helvetica; font-size: 9pt; }.quote { margin-left: 1em; margin-right: 1em; border-left: 5px #ebebeb solid; padding-left: 0.3em; }a.em-mention[href] { text-decoration: none; color: inherit; border-radius: 3px; padding-left: 2px; padding-right: 2px; background-color: #e2e2e2; }._em_placeholder {color: gray; border-bottom: 1px dotted lightblue;}._em_placeholder:before{color:gray; content: '{{ ';}._em_placeholder:after{color:gray; content: ' }}';}</style>
in <a href="https://bugs.openjdk.org/browse/JDK-8344697" moz-do-not-send="true" class="moz-txt-link-freetext">https://bugs.openjdk.org/browse/JDK-8344697</a> I
pointed out that in Mac's Aqua L&F: we’re using
AncestorListeners to help repaint JRootPane default buttons.
<div><br>
</div>
<div>Over a decade ago this logic was needed to emulate the native
Aqua experience, but since around 2016 the default button in
Aqua does NOT continually pulse.</div>
<div><br>
</div>
<div>I’d like to remove this code, but I’m not sure how to
proceed.</div>
<div><br>
</div>
<div>Originally I planned to make a PR for <a href="https://bugs.openjdk.org/browse/JDK-8344697" style="font-size: 9pt;" moz-do-not-send="true">8344697</a> that
removes the unused code. But after rebalancing the
AWTEventMulticaster (in <a href="https://bugs.openjdk.org/browse/JDK-8342782" style="font-size: 9pt;" moz-do-not-send="true" class="moz-txt-link-freetext">https://bugs.openjdk.org/browse/JDK-8342782</a> ),
the performance problems observed in<span> </span><a href="https://bugs.openjdk.org/browse/JDK-8344697" style="font-size: 9pt; margin: 0px;" moz-do-not-send="true">8344697</a> do
not reproduce.</div>
<div><br>
</div>
<div>Should I:</div>
<div>A. File a new ticket just to remove the obsolete code?</div>
<div>B. Go ahead and name this new PR after<span> </span><a href="https://bugs.openjdk.org/browse/JDK-8344697" style="font-size: 9pt; margin: 0px;" moz-do-not-send="true">8344697</a> ,
even though that’s a little misleading now?</div>
<div>C. Don’t pursue this, because without a concrete problem /
business case the OpenJDK community shouldn’t focus on this tech
debt.</div>
<div><br>
</div>
<div>Thoughts?</div>
<div><br>
</div>
<div>Regards,</div>
<div> - Jeremy</div>
</blockquote>
<br>
</body>
</html>