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