On Mon, 29 Sep 2025 23:36:00 GMT, Henry Jen <[email protected]> wrote:
> Since parameters is an opt-in choice, it's more reasonable to consider that's > desired information and make strip parameter names an opt-in choice as well. > > This PR changes the default behavior of --strip-debug to keep parameter names > when it's available. Add opt-in mechanism, > via the strip-java-debug-attributes plugin by using argument > `--strip-java-debug-attributes=+parameter-names`. > > The --strip-debug option is a little bit odd, as it's a main option as well > as a plugin option to enable the DefaultStripDebugPlugin, which strip native > debug information on platform support the feature, and strip java debug > information. In this PR, we chose to support only one mechanism to enable > strip parameter names, so we would disable the embed > StripJavaDebugAttributesPlugin when StripJavaDebugAttributesPlugin is enabled. > > The StripParameterNamesTest illustrate and verify parameter names use cases, > mainly focus on argument processing and the parameter names. We didn't verify > the regular debug info as that's covered by existing test. > > -- Update > > The latest change removed the support of strip parameter names, as the > MethodParameters attribute is considered necessary attribute to fulfill Java > Language Spec even though this attribute is considered optional by JVM spec. This pull request has now been integrated. Changeset: b83e7939 Author: Henry Jen <[email protected]> URL: https://git.openjdk.org/jdk/commit/b83e7939dd9401314f2351a9fefd1e212387c6e5 Stats: 296 lines in 5 files changed: 289 ins; 2 del; 5 mod 8347007: --strip-debug removes parameter names included with -parameters Reviewed-by: alanb ------------- PR: https://git.openjdk.org/jdk/pull/27566
