Iain, I tried it in my local test suite, and everything worked.
So, it looks good to me. You'll probably notice that you misspelled
"system" as "sytem", so I won't mention it.
> -----Original Message-----
> From: Iain Sandoe <iains....@gmail.com>
> Sent: Monday, March 17, 2025 19:52
> To: rdub...@symas.com; gcc-patches@gcc.gnu.org
> Subject: [PATCH] cobol: Do not include C++ headers after system.h.
>
> tested on x86_64-darwin, linux and aarch64-linux,
> OK for trunk?
> thanks
> Iain
>
> --- 8< ---
>
> The headers mentioned in parse.y are already unconditionally included
> by system.h (via cobol-system.h).
>
> gcc/cobol/ChangeLog:
>
> * parse.y: Remove c++ header includes appearing after
> sytem.h.
>
> Signed-off-by: Iain Sandoe <i...@sandoe.co.uk>
> ---
> gcc/cobol/parse.y | 6 ------
> 1 file changed, 6 deletions(-)
>
> diff --git a/gcc/cobol/parse.y b/gcc/cobol/parse.y
> index c901bd66629..d8f5175adcb 100644
> --- a/gcc/cobol/parse.y
> +++ b/gcc/cobol/parse.y
> @@ -30,9 +30,6 @@
> %code requires {
> #include <fstream> // Before cobol-system because it uses poisoned
> functions
> #include "cobol-system.h"
> - #include <cmath>
> - #include <algorithm>
> - #include <map>
> #include "../../libgcobol/io.h"
> #include "../../libgcobol/ec.h"
>
> @@ -216,9 +213,6 @@
> new_literal( const char initial[], enum radix_t radix );
> #pragma GCC diagnostic pop
>
> -
> - #include <list>
> -
> enum select_clause_t {
> access_clause_e = 0x0001,
> alt_key_clause_e = 0x0002,
> --
> 2.39.2 (Apple Git-143)
