<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="">Hi Conor,<div class=""><br class=""></div><div class="">Overall I think this is OK.  A few comments to consider:</div><div class=""><br class=""><div class=""><br class=""></div><div class="">Have you created a CSR for this change yet?  If not you will need one.</div><div class=""><br class=""></div><div class="">The description for almost all of the constructors indicate:</div><div class=""><br class=""></div><div class="">————</div><div class=""><pre class="">Constructor for subclasses to call</pre></div><div class="">——————</div><div class=""><br class=""></div><div class="">Is the above wording used elsewhere in the JDK?  Not sure I like it, I might suggest  a little wordsmithing</div><div class=""><br class=""></div><div class="">For example AbstractList uses:</div><div class=""><br class=""></div><div class="">—————</div><div class=""><span style="caret-color: rgb(71, 71, 71); color: rgb(71, 71, 71); font-family: "DejaVu Serif", Georgia, "Times New Roman", Times, serif; font-size: 14px; background-color: rgb(255, 255, 255);" class="">Sole constructor. (For invocation by subclass constructors, typically implicit.)</span></div><div class="">--------------</div><div class=""><br class=""></div><div class="">For ImageFilter (as an example):</div><div class=""><br class=""></div><div class="">———</div><div class=""><pre class="">Creates an {@code ImageFilter}</pre></div><div class="">--------</div><div class=""><div><br class=""></div><div>I would probably tweak it to something similar to:</div><div><br class=""></div><div>————</div><div><span style="caret-color: rgb(71, 71, 71); color: rgb(71, 71, 71); font-family: "DejaVu Serif", Georgia, "Times New Roman", Times, serif; font-size: 14px; background-color: rgb(255, 255, 255);" class="">HashTable: Constructs a new, empty hashtable </span></div><div><br class=""></div><div>FileSystem: <span style="caret-color: rgb(71, 71, 71); color: rgb(71, 71, 71); font-family: "DejaVu Serif", Georgia, "Times New Roman", Times, serif; font-size: 14px; background-color: rgb(255, 255, 255);" class="">Initializes a new instance of this class.</span></div><div>—————</div><div><br class=""></div><div>I would suggest including “new” in your proposed wording at a minimum</div><div><br class=""></div><div>Best</div><div>Lance</div><div><br class=""></div><div><blockquote type="cite" class=""><div class="">On Aug 17, 2020, at 6:11 AM, Conor Cleary <<a href="mailto:conor.cleary@oracle.com" class="">conor.cleary@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi all,<br class=""><br class="">Looking for reviewers for JDK-8250855, 'Address reliance on default constructors in the Java 2D APIs'.<br class=""><br class="">This patch addresses the reliance on default constructors in the following packages:<br class=""><br class=""> * java.awt.Image<br class=""> * java.awt.PrintJob<br class=""> * java.awt.font.GlyphVector<br class=""> * java.awt.font.LayoutPath<br class=""> * java.awt.font.LineMetrics<br class=""> * java.awt.image.AbstractMultiResolutionImage<br class=""> * java.awt.image.BufferStrategy<br class=""> * java.awt.image.ImageFilter<br class=""> * java.awt.image.RGBImageFilter<br class=""> * java.awt.image.VolatileImage<br class=""> * javax.print.PrintServiceLookup<br class=""> * javax.print.ServiceUI<br class=""> * javax.print.ServiceUIFactory<br class=""> * javax.print.StreamPrintServiceFactory<br class=""> * javax.print.event.PrintJobAdapter<br class=""><br class="">The patch places simple constructors in all of these packages.<br class=""><br class=""> * bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8250855" class="">https://bugs.openjdk.java.net/browse/JDK-8250855</a><br class=""> * webrev:<br class="">   <a href="http://cr.openjdk.java.net/~ccleary/issues/webrevs-store/8250855/webrevs/webrev.00/" class="">http://cr.openjdk.java.net/~ccleary/issues/webrevs-store/8250855/webrevs/webrev.00/</a><br class=""><br class=""><br class="">Conor<br class=""><br class=""></div></div></blockquote></div><br class=""><div class="">
<div style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;" class=""><span><br class="">Best<br class="">Lance<br class=""></span></div><span style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;"><div class=""><span>------------------</span></div></span><br class="Apple-interchange-newline"><span><img apple-inline="yes" id="0CED2E8E-C53A-4EF1-B377-CC32D189B4E5" src="cid:E1C4E2F0-ECD0-4C9D-ADB4-B16CA7BCB7FC@home" class=""></span><div style="font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none; caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);"><br class="Apple-interchange-newline"><br class="">Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037<br class="">Oracle Java Engineering <br class="">1 Network Drive <br class="">Burlington, MA 01803<br class=""><a href="mailto:Lance.Andersen@oracle.com" class="">Lance.Andersen@oracle.com</a></div><div style="font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none; caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0);" class=""><br class=""></div><br class="Apple-interchange-newline" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); font-family: Helvetica; font-size: 18px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; text-decoration: none;"><br class="Apple-interchange-newline">
</div>
<br class=""></div></div></body></html>