RFR: 8352645: Add tool support to check order of includes [v3]
Stefan Karlsson
stefank at openjdk.org
Thu Mar 27 11:16:07 UTC 2025
On Thu, 27 Mar 2025 10:36:38 GMT, Doug Simon <dnsimon at openjdk.org> wrote:
> > A couple of nits:
> >
> > 1. jcheck fails because of whitespaces
> > 2. The /// style comments is a style I haven't encountered before.
>
> I fixed the whitespaces. I can convert the `///` comments if you want - no strong opinion.
Maybe someone else knows the preferred style for this? I don't think we need to block the integration because of this. If someone comes late with the proper comment style, we'll update it in a separate PR.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/24247#issuecomment-2757648203
More information about the hotspot-dev
mailing list