Cleans final production mode warnings from H5Shyper.c (#2163)

* H5S__hyper_free_span_info() returns herr_t (not checked)

* Set up H5S__hyper_free_span_info() error handling

* Set up H5S__hyper_free_span() error handling

* Added error handling to H5S_HYPER_ADVANCE_SPAN

* Handled errors in H5S__hyper_iter_get_seq_list_gen()

* Added a production mode build target for -Werror

* Committing clang-format changes

* Updated names for -Werror checks

Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>

.github/workflows/main.yml

@@ -269,7 +269,7 @@ jobs:
                 cmake: "Debug"
                 autotools: "debug"
 
-          - name: "Ubuntu gcc Autotools -Werror (build only)"
+          - name: "Ubuntu gcc Autotools -Werror (build only) DBG"
             os: ubuntu-latest
             cpp: enable
             fortran: disable
@@ -291,6 +291,28 @@ jobs:
                 cmake: "Debug"
                 autotools: "debug"
 
+          - name: "Ubuntu gcc Autotools -Werror (build only) REL"
+            os: ubuntu-latest
+            cpp: enable
+            fortran: disable
+            java: disable
+            parallel: disable
+            mirror_vfd: enable
+            direct_vfd: enable
+            deprec_sym: enable
+            default_api: v114
+            toolchain: ""
+            generator: "autogen"
+            flags: "CFLAGS=-Werror"
+            run_tests: false
+            thread_safety:
+              - enabled: false
+                text: ""
+            build_mode:
+              - text: "REL"
+                cmake: "Release"
+                autotools: "production"
+
     # Sets the job's name from the properties
     name: "${{ matrix.name }}${{ matrix.build_mode.text }}${{ matrix.thread_safety.text }}"