RFR: 8352645: Add tool support to check order of includes [v2]

Stefan Karlsson stefank at openjdk.org
Thu Mar 27 08:46:10 UTC 2025


On Thu, 27 Mar 2025 06:10:37 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Doug Simon has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   drop extra blank lines and preserve rule for first include in .inline.hpp files
>
> src/hotspot/share/compiler/disassembler.hpp line 36:
> 
>> 34: #include "utilities/macros.hpp"
>> 35: 
>> 36: 
> 
> Extra blank line inserted?

This seems to be left-overs from an earlier run. If I run the tool on this file it doesn't add this blank line.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24247#discussion_r2015915647


More information about the hotspot-dev mailing list