RFR: 8322936: Update blessed-modifier-order.sh for default, sealed, and non-sealed

Martin Buchholz martin at openjdk.org
Thu Jan 4 07:18:29 UTC 2024


On Wed, 3 Jan 2024 17:39:56 GMT, Pavel Rappo <prappo at openjdk.org> wrote:

>> bin/blessed-modifier-order.sh line 34:
>> 
>>> 32:     echo
>>> 33:     echo "See:"
>>> 34:     echo "https://docs.oracle.com/javase/8/docs/api/java/lang/reflect/Modifier.html#toString-int-"
>> 
>> Looking at the latest docs for Modifier#toString()
>> it also needs a corresponding update
>> https://download.java.net/java/early_access/jdk23/docs/api/java.base/java/lang/reflect/Modifier.html#toString(int)
>> 
>> and if that is done, then we can keep referring to its javadoc.
>> 
>> References to javadoc and jls both have problems of becoming stale over time. 
>> There is no way to reference the "latest" spec via a URL.
>
>> it also needs a corresponding update
>> https://download.java.net/java/early_access/jdk23/docs/api/java.base/java/lang/reflect/Modifier.html#toString(int)
> 
> From the preceding discussion, it follows that it cannot be done: https://mail.openjdk.org/pipermail/core-libs-dev/2024-January/117381.html

Thanks ... now I see ...

---

When I originally wrote this script I recall hitting some false positives - a price we pay for having this modify comments.  Don't blindly apply all suggestions!

---

I don't recall considering "default".

---

It seems we no longer have a single canonical reference for modifier order.
(or more generally, a well-maintained high quality Java program style guide ...)

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/17242#discussion_r1441411821


More information about the build-dev mailing list