fixes #1064 Potential deadlock in statistics code

fixes #1063 Include sanitizer runs in CI
fixes #1068 Wssfile test sometimes fails with wrong error code

While here, addressed a number of clang-tidy items, and some light
cleanup of code we were already in.

1 files changed, 35 insertions, 0 deletions

diff --git a/.github/workflows/sanitizer.yml b/.github/workflows/sanitizer.yml
new file mode 100644
index 00000000..e55ed227
--- /dev/null
+++ b/.github/workflows/sanitizer.yml
@@ -0,0 +1,35 @@
+name: sanitize
+on: [push]
+jobs:
+  sanitize:
+    env:
+      CC: clang
+      CXX: clang++
+      CTEST_OUTPUT_ON_FAILURE: 1
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        sanitizer: [ address, undefined, thread ]
+        os: [ ubuntu-latest ]
+
+    steps:
+      - uses: actions/checkout@v1
+
+      - uses: mstksg/get-package@v1
+        with:
+            apt-get: ninja-build libmbedtls-dev
+            brew: ninja mbedtls
+
+      - name: Configure
+        run: |
+          mkdir build
+          cd build
+          cmake -G Ninja -DNNG_SANITIZER=${{ matrix.sanitizer }} -DNNG_ENABLE_TLS=ON -DNNG_TOOLS=OFF ..
+      - name: Build
+        run: |
+          cd build
+          ninja
+      - name: Test
+        run: |
+          cd build
+          ninja test
\ No newline at end of file