On Mon, 14 Nov 2022 09:25:11 GMT, Stefan Karlsson <stef...@openjdk.org> wrote:
> One of the more prevalent issues is that files in src/hotspot/share/include > are not properly sorted. There has been some discussion that that was done on > purpose, but it just adds another exception to the include rules that don't > have any practical purposes, IMHO. It also goes against our written style > guide around include files. One argument why it was OK have the files in > include/ pushed up to the top of the sorted block, was that the file was > included without specifying a directory. That's an argument that contradicts > how we treat platform-dependent files, which (unfortunately) often also are > specified without a prefixed directory. To remove this special case, I've > removed the extraneous make file entry to have src/hotspot/share/include in > the set of directories to search for headers when compiling HotSpot. Now all > the header files in src/hotspot/share/include gets included by specifying the > path from src/hotspot/share, just like the other platform-independent headers > in HotSpot . > > This RFE splits out the 'include/' changes from #11108 / JDK-8296886, so that > those changes can be discussed separately. This pull request has now been integrated. Changeset: 813b223a Author: Stefan Karlsson <stef...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/813b223a6bcd9f6290ee9c8840a8c69061ade48c Stats: 231 lines in 113 files changed: 111 ins; 117 del; 3 mod 8296926: Sort include lines of files in the include/ directory Reviewed-by: kbarrett, erikj, lkorinth ------------- PR: https://git.openjdk.org/jdk/pull/11133
