RFR: 8356390: Rename ResolvedIndyEntry::set_flags to set_has_appendix [v3]

Johan Sjölen jsjolen at openjdk.org
Fri May 9 12:19:33 UTC 2025


> The `set_flags` function really only sets whether it has an appendix or not, and there's a separate `set_resolution_failed` method just below that also alters the flag. Just rename this to `set_has_appendix`

Johan Sjölen has updated the pull request incrementally with two additional commits since the last revision:

 - We missed the case of has_appendix = false
 - And make the code uniform for both flags

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25092/files
  - new: https://git.openjdk.org/jdk/pull/25092/files/beccef03..4da8d47b

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25092&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25092&range=01-02

  Stats: 7 lines in 1 file changed: 3 ins; 1 del; 3 mod
  Patch: https://git.openjdk.org/jdk/pull/25092.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25092/head:pull/25092

PR: https://git.openjdk.org/jdk/pull/25092


More information about the hotspot-dev mailing list