Re: RFR: 8364115: Sort share/services includes [v2]

2025-07-30 Thread Aleksey Shipilev
On Mon, 28 Jul 2025 14:41:25 GMT, Francesco Andreuzzi wrote: >> This PR sorts the includes in `hotspot/share/services` using >> `SortIncludes.java`. I'm also adding the directory to >> `TestIncludesAreSorted`. >> >> Passes tier1. > > Francesco Andreuzzi has updated the pull request with a new

Re: RFR: 8364115: Sort share/services includes [v2]

2025-07-30 Thread duke
On Mon, 28 Jul 2025 14:41:25 GMT, Francesco Andreuzzi wrote: >> This PR sorts the includes in `hotspot/share/services` using >> `SortIncludes.java`. I'm also adding the directory to >> `TestIncludesAreSorted`. >> >> Passes tier1. > > Francesco Andreuzzi has updated the pull request with a new

Re: RFR: 8364115: Sort share/services includes [v2]

2025-07-30 Thread Francesco Andreuzzi
On Wed, 30 Jul 2025 07:15:45 GMT, Albert Mingkun Yang wrote: >> Francesco Andreuzzi has updated the pull request with a new target base due >> to a merge or a rebase. The pull request now contains two commits: >> >> - Merge branch 'master' into JDK-8364115 >> - sort includes > > Marked as rev

Re: RFR: 8364115: Sort share/services includes [v2]

2025-07-30 Thread Albert Mingkun Yang
On Mon, 28 Jul 2025 14:41:25 GMT, Francesco Andreuzzi wrote: >> This PR sorts the includes in `hotspot/share/services` using >> `SortIncludes.java`. I'm also adding the directory to >> `TestIncludesAreSorted`. >> >> Passes tier1. > > Francesco Andreuzzi has updated the pull request with a new

Re: RFR: 8364115: Sort share/services includes [v2]

2025-07-29 Thread Serguei Spitsyn
On Mon, 28 Jul 2025 14:41:25 GMT, Francesco Andreuzzi wrote: >> This PR sorts the includes in `hotspot/share/services` using >> `SortIncludes.java`. I'm also adding the directory to >> `TestIncludesAreSorted`. >> >> Passes tier1. > > Francesco Andreuzzi has updated the pull request with a new

Re: RFR: 8364115: Sort share/services includes [v2]

2025-07-28 Thread Francesco Andreuzzi
> This PR sorts the includes in `hotspot/share/services` using > `SortIncludes.java`. I'm also adding the directory to `TestIncludesAreSorted`. > > Passes tier1. Francesco Andreuzzi has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains two

RFR: 8364115: Sort share/services includes

2025-07-25 Thread Francesco Andreuzzi
This PR sorts the includes in `hotspot/share/services` using `SortIncludes.java`. I'm also adding the directory to `TestIncludesAreSorted`. Passes tier1. - Commit messages: - sort includes Changes: https://git.openjdk.org/jdk/pull/26483/files Webrev: https://webrevs.openjdk.org/