RFR: 8278143: Remove unused "argc" from ConstantPool::copy_bootstrap_arguments_at_impl

David Holmes dholmes at openjdk.java.net
Fri Dec 3 04:50:22 UTC 2021


On Thu, 2 Dec 2021 12:49:45 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:

> SonarCloud complains about variable initialization in large conditional statements. In that tower of if-expressions, it is not even clear to me that `argc` would be properly initialized. Seems better to clean that up.
> 
> Additional testing:
>  - [x] Linux x86_64 fastdebug build
>  - [ ] GHA

Good catch! :) It has been unused since it was introduced by JDK-8186209

argc is still mentioned in the comment on line 1183.

Thanks,
David

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/6671


More information about the hotspot-dev mailing list