I'm going to take your word on this one. LGTM
> -----Original Message----- > From: Iain Sandoe <iains....@gmail.com> > Sent: Monday, March 24, 2025 06:21 > To: jklow...@cobolworx.com; rdub...@symas.com; gcc-patches@gcc.gnu.org > Subject: [PATCH] cobol: Move includes before system.h > > A trivial patch that ensures host includes referenced directly are > included before system.h (to avoid issues with poisoned interfaces). > Tested on x86_64-linux,darwin aarch64-linux, OK for trunk? > thanks > Iain > > --- 8< --- > > This just moves a couple of includes ahead of cobol-system.h which > in turn includes system.h. > > gcc/cobol/ChangeLog: > > * cdf-copy.cc: Move host includes before system.h > > Signed-off-by: Iain Sandoe <i...@sandoe.co.uk> > --- > gcc/cobol/cdf-copy.cc | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/gcc/cobol/cdf-copy.cc b/gcc/cobol/cdf-copy.cc > index 179dbacea93..c620c828a0b 100644 > --- a/gcc/cobol/cdf-copy.cc > +++ b/gcc/cobol/cdf-copy.cc > @@ -34,13 +34,13 @@ > // > // We regret any confusion engendered. > > +#include <glob.h> > + > #include "cobol-system.h" > #include "cbldiag.h" > #include "util.h" > #include "copybook.h" > > -#include <glob.h> > - > #define COUNT_OF(X) (sizeof(X) / sizeof(X[0])) > > /* > -- > 2.39.2 (Apple Git-143)
