On Wed, 23 Nov 2022 07:12:13 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:

> > I'm not sure why conditional includes (that don't rely on macros.hpp) need 
> > to come at the end rather than in normal sort order? I don't care either 
> > way but a rationale for this would be good if it is to be the preferred 
> > style.
> 
> Because the Style Guide says:
> 
>     * Put conditional inclusions (`#if ...`) at the end of the include list.

Ah I see. Thanks for that @kimbarrett  
> 
> I think most of our conditional includes these days are to support 
> conditional features. It makes sense to group all the additional includes 
> related to a feature.

True. There are a lot of single includes but it makes sense to have one simple 
rule.

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

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

Reply via email to