Browse Source

Merge pull request #899 from paullouisageneau/fix-workflow-warning-as-errors

Fix warnings as errors flag in workflow
Paul-Louis Ageneau 2 years ago
parent
commit
92479f4a52
3 changed files with 3 additions and 3 deletions
  1. 1 1
      .github/workflows/build-nomedia.yml
  2. 1 1
      .github/workflows/build-openssl.yml
  3. 1 1
      src/capi.cpp

+ 1 - 1
.github/workflows/build-nomedia.yml

@@ -29,7 +29,7 @@ jobs:
     - name: submodules
     - name: submodules
       run: git submodule update --init --recursive --depth 1
       run: git submodule update --init --recursive --depth 1
     - name: cmake
     - name: cmake
-      run: cmake -B build -G "NMake Makefiles" -DUSE_GNUTLS=0 -DNO_MEDIA=1 -WARNINGS_AS_ERRORS=1
+      run: cmake -B build -G "NMake Makefiles" -DUSE_GNUTLS=0 -DNO_MEDIA=1 -DWARNINGS_AS_ERRORS=1
     - name: nmake
     - name: nmake
       run: |
       run: |
         cd build
         cd build

+ 1 - 1
.github/workflows/build-openssl.yml

@@ -45,7 +45,7 @@ jobs:
     - name: submodules
     - name: submodules
       run: git submodule update --init --recursive --depth 1
       run: git submodule update --init --recursive --depth 1
     - name: cmake
     - name: cmake
-      run: cmake -B build -G "NMake Makefiles" -DUSE_GNUTLS=0 -WARNINGS_AS_ERRORS=1
+      run: cmake -B build -G "NMake Makefiles" -DUSE_GNUTLS=0 -DWARNINGS_AS_ERRORS=1
     - name: nmake
     - name: nmake
       run: |
       run: |
         cd build
         cd build

+ 1 - 1
src/capi.cpp

@@ -309,7 +309,7 @@ public:
 			return msg;
 			return msg;
 		}
 		}
 
 
-		auto res = incomingCallback(reinterpret_cast<void *>(msg->data()), msg->size());
+		auto res = incomingCallback(reinterpret_cast<void *>(msg->data()), int(msg->size()));
 
 
 		// If a null pointer was returned, drop the incoming message
 		// If a null pointer was returned, drop the incoming message
 		if (res == nullptr) {
 		if (res == nullptr) {