From 345ed66fabd5e7550e82e0a213caf6d382c71c85 Mon Sep 17 00:00:00 2001
From: matt335672 <30179339+matt335672@users.noreply.github.com>
Date: Mon, 11 Oct 2021 20:01:10 +0100
Subject: Add extra compiler and separate out cache stage

---
 .github/workflows/build.yml | 35 +++++++++++++++++++++++++++++++++--
 1 file changed, 33 insertions(+), 2 deletions(-)

diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
index 20c661c..5090b81 100644
--- a/.github/workflows/build.yml
+++ b/.github/workflows/build.yml
@@ -4,8 +4,8 @@ on:
   pull_request:
 
 jobs:
-  build:
-    name: build
+  install_pa_sources:
+    name: install PA sources
     runs-on: ubuntu-latest
     steps:
       # This is currently the only way to get a version into
@@ -21,6 +21,37 @@ jobs:
           path: ~/pulseaudio.src
           key: ${{ runner.os }}-${{ env.OS_VERSION }}-build-${{ env.cache-name }}
       - run: scripts/install_pulseaudio_sources.sh
+
+  build:
+    strategy:
+      fail-fast: false
+      matrix:
+        include:
+
+          - CC: gcc
+
+          - CC: clang
+
+    name: build with ${{ matrix.CC }}
+    runs-on: ubuntu-latest
+    needs: install_pa_sources
+    env:
+      CC: ${{ matrix.CC }}
+      CFLAGS: -Wall -Wextra -Werror
+
+    steps:
+      # This is currently the only way to get a version into
+      # the cache tag name - see https://github.com/actions/cache/issues/543
+      - run: |
+          echo "OS_VERSION=`lsb_release -sr`" >> $GITHUB_ENV
+      - name: Fetch pulseaudio sources
+        uses: actions/cache@v2
+        env:
+          cache-name: cache-pulseaudio-src
+        with:
+          path: ~/pulseaudio.src
+          key: ${{ runner.os }}-${{ env.OS_VERSION }}-build-${{ env.cache-name }}
+      - uses: actions/checkout@v2
       - run: sudo apt-get update
       - run: sudo apt-get -yq install build-essential libpulse-dev
       - run: ./bootstrap
-- 
cgit v1.2.3


From e447bc4cffc02621ac66b48eb34544d9aeaa6ea1 Mon Sep 17 00:00:00 2001
From: matt335672 <30179339+matt335672@users.noreply.github.com>
Date: Mon, 11 Oct 2021 20:30:53 +0100
Subject: Fix extra compiler warnings

---
 src/module-xrdp-sink.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/module-xrdp-sink.c b/src/module-xrdp-sink.c
index 62f92d9..71b20fd 100644
--- a/src/module-xrdp-sink.c
+++ b/src/module-xrdp-sink.c
@@ -372,8 +372,6 @@ static void process_render(struct userdata *u, pa_usec_t now) {
 static void thread_func(void *userdata) {
 
     struct userdata *u = userdata;
-    int ret;
-    pa_usec_t now;
 
     pa_assert(u);
 
-- 
cgit v1.2.3