On Wed, 11 Feb 2026 09:38:15 GMT, Larry Cable <[email protected]> wrote:
>> modified the pre-existing VM.classes jcmd to add a 'location' option, that >> when specified, will (natively) attempt to obtain the value (if non-null) of >> the location URL of the CodeSource of each classes ProtectionDomain. >> >> effectively: >> >> someObject.getClass().getProtectionDomain().getCodeSource().getLocation().toExternalForm() >> >> (where interim oops are null-checked) > > Larry Cable has updated the pull request incrementally with two additional > commits since the last revision: > > - JDK-8327246: remove use of R"(...)" raw string const > - JDK-8327246: fixed jcmd -help text and documentation to correctly document > class flag annotationso as independent of -verbose option Nearly done. There are whitespace errors in src/jdk.jcmd/share/man/jcmd.md and it doesn't match the new ClassesDCmd desc string. Personally I like "Print all loaded classes." with the full stop and just "Classes may be annotated with flags.." as the current "Some classes" is really mysterious! Thanks for making this help readable for the first time. CSR looks good also. It could say that -location is a BOOLEAN argument, default false. And maybe that it affects the new flags which can be shown. I really like it when JBS issues or PRs (or both) include example output so people know what they're getting. ------------- PR Review: https://git.openjdk.org/jdk/pull/29048#pullrequestreview-3809161593
