RFR: 8278143: Remove unused "argc" from ConstantPool::copy_bootstrap_arguments_at_impl [v2]
Aleksey Shipilev
shade at openjdk.java.net
Fri Dec 3 08:23:41 UTC 2021
> 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
Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
Tidy up the comment
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/6671/files
- new: https://git.openjdk.java.net/jdk/pull/6671/files/18fba172..a2718459
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=6671&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=6671&range=00-01
Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/6671.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/6671/head:pull/6671
PR: https://git.openjdk.java.net/jdk/pull/6671
More information about the hotspot-dev
mailing list