RFR: 8323158: HotSpot Style Guide should specify more include ordering [v2]

Stefan Karlsson stefank at openjdk.org
Mon Feb 3 12:14:35 UTC 2025


> The HotSpot Style Guide has a section about source files and includes. The style used for includes have mostly been introduced by scripts when includeDB was replaced, but also when various other enhancements to our includes were made. Some of the introduced styles were never written down in the style guide.
> 
> I propose a couple of changes to the HotSpot Style Guide to reflect some of these implicit styles that we have. While updating the text I also took the liberty to order the items in an order that I felt was good.
> 
> Note that JDK-8323158 contains a few more suggestions, but I've only addressed the items that I think can be accepted without much contention. Either I extract the items that have not been address into a new RFE, or I create a new RFE for this PR.
> 
> There a some small whitespace tweaks that I made so that the .md and .html had a similar layout.

Stefan Karlsson has updated the pull request incrementally with two additional commits since the last revision:

 - Update hotspot-style.md
 - Update hotspot-style.html

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/23388/files
  - new: https://git.openjdk.org/jdk/pull/23388/files/51913afa..f01c564c

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

  Stats: 4 lines in 2 files changed: 0 ins; 0 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/23388.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/23388/head:pull/23388

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


More information about the build-dev mailing list