RFR: 8354309: Sort GC includes [v2]
Stefan Karlsson
stefank at openjdk.org
Wed Apr 23 10:10:20 UTC 2025
> We now have a tool to fix our include lines to better adhere to the HotSpot Style Guide. See test/hotspot/jtreg/sources/SortIncludes.java. I've run the tool against the GC code and propose the resulting changes. I did one manual change to move a section guarded with INCLUDE_JFR.
>
> Builds successfully with GHA. I will run this through tier1-2.
Stefan Karlsson has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision:
- Fix windows compliation error
- Merge remote-tracking branch 'upstream/master' into 8354309_sort_gc_includes
- Merge remote-tracking branch 'upstream/master' into 8354309_sort_gc_includes
- 8354309: Sort GC includes
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/24581/files
- new: https://git.openjdk.org/jdk/pull/24581/files/cf81077d..54a5d61b
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=24581&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=24581&range=00-01
Stats: 214494 lines in 1181 files changed: 38563 ins; 172575 del; 3356 mod
Patch: https://git.openjdk.org/jdk/pull/24581.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/24581/head:pull/24581
PR: https://git.openjdk.org/jdk/pull/24581
More information about the hotspot-dev
mailing list