More fixes

.github/workflows/windows-msvc.yml

@@ -204,6 +204,7 @@ jobs:
 
       - name: Upload testsuite-${{ matrix.arch }}-${{ matrix.target }}.log
         uses: actions/upload-artifact@v4
+        if: failure()
         with:
           name: testsuite-${{ matrix.arch }}-${{ matrix.target }}.log
           path: ${{ env.GITHUB_WORKSPACE }}/tests/testsuite.log

.github/workflows/windows-msys1.yml

@@ -229,6 +229,7 @@ jobs:
 
       - name: Upload testsuite-${{ matrix.target }}.log
         uses: actions/upload-artifact@v4
+        if: failure()
         with:
           name: testsuite-${{ matrix.target }}.log
           path: ${{ env.GITHUB_WORKSPACE }}/_build/tests/testsuite.log

.github/workflows/windows-msys2.yml

@@ -102,6 +102,7 @@ jobs:
 
       - name: Upload testsuite-${{matrix.sys}}-${{matrix.target}}.log
         uses: actions/upload-artifact@v4
+        if: failure()
         with:
           name: testsuite-${{matrix.sys}}-${{matrix.target}}.log
           path: ${{ env.GITHUB_WORKSPACE }}/_build/tests/testsuite.log

cobc/ChangeLog

@@ -1,4 +1,8 @@
 
+2024-09-23  David Declerck <[email protected]>
+
+	* pplex.l (cb_text_list): prevent duplicates
+
 2024-08-28  David Declerck <[email protected]>
 
 	* tree.c (char_to_precedence_idx, get_char_type_description, valid_char_order):
@@ -116,16 +120,13 @@
 
 2024-03-15  Fabrice Le Fessant <[email protected]>
 
-	* cobc.c: add new flags to output dependencies following gcc: -M to
-	  output deps only, -MD to output deps while compiling (in .d files),
-	  -MP to output phony targets, -MG to keep missing copybooks,
-	  -MQ <target> to Makefile-quote target
-	* flags.def: -fcopybook-deps outputs only copybook names instead
-	  of file paths. -fcopybook-deps also forces -E, -foneline-deps,
-	  -MT=copybooks, disables errors on missing copybooks and removes
-	  output on stdout.
-        * typeck.c (check_argument_conformance): check that param is well defined
-	  to prevent a segfault
+	* cobc.c, help.c, pplex.l: add new flags to output dependencies following
+	  gcc: -M to output deps only, -MD to output deps while compiling (in
+	  .d files), -MP to output phony targets, -MG to keep missing copybooks,
+	  -MQ <target> to Makefile-quote target ; add -fcopybook-deps to
+	  output only copybook names instead of file paths. -fcopybook-deps
+	  also forces -E, -foneline-deps, -MT=copybooks, disables errors on
+	  missing copybooks and removes output on stdout
 
 2024-02-19  Boris Eng <[email protected]>
 

cobc/typeck.c

@@ -3588,7 +3588,6 @@ check_argument_conformance (struct cb_program *program, cb_tree argument_tripple
 	} else {
 		arg_field = NULL;
 	}
-	if (!CB_FIELD_P(param)) return;
 	param_field = CB_FIELD_PTR(CB_VALUE(param));
 
 	/*