diff options
| author | Even Rouault <even.rouault@spatialys.com> | 2021-11-09 12:43:13 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-11-09 12:43:13 +0100 |
| commit | c20b3f6d1beaa7c5f1325d42e0f244ebaea52455 (patch) | |
| tree | 87c8982a4c0d1b41bf5b47c5271cb6791669ad32 | |
| parent | fa0c0e617e4fdc5bf1e2bb2142a4213698808155 (diff) | |
| parent | f7e9db583b709b62aabeaf8ae7f8b1f7e1505395 (diff) | |
| download | PROJ-c20b3f6d1beaa7c5f1325d42e0f244ebaea52455.tar.gz PROJ-c20b3f6d1beaa7c5f1325d42e0f244ebaea52455.zip | |
Merge pull request #2932 from rouault/fix_2931
PROJStringFormatter::toString(): avoid invalid iterator increment (fixes #2931)
| -rw-r--r-- | .github/workflows/windows.yml | 14 | ||||
| -rw-r--r-- | src/iso19111/io.cpp | 2 |
2 files changed, 11 insertions, 5 deletions
diff --git a/.github/workflows/windows.yml b/.github/workflows/windows.yml index 4425fccd..9358aaa6 100644 --- a/.github/workflows/windows.yml +++ b/.github/workflows/windows.yml @@ -8,9 +8,15 @@ jobs: runs-on: windows-latest if: "!contains(github.event.head_commit.message, '[ci skip]') && !contains(github.event.head_commit.message, '[skip ci]')" + strategy: + fail-fast: false + matrix: + BUILD_TYPE: [Release, Debug] + env: ARCH: x64 BUILD_SHARED_LIBS: ON + BUILD_TYPE: ${{ matrix.BUILD_TYPE }} steps: @@ -42,7 +48,7 @@ jobs: mkdir %PROJ_BUILD% cd %PROJ_BUILD% set PROJ_DIR=%GITHUB_WORKSPACE%\proj_dir - cmake -GNinja .. -DCMAKE_BUILD_TYPE=Release -DBUILD_SHARED_LIBS="{{ env.BUILD_SHARED_LIBS }}" -DCMAKE_C_FLAGS="/WX" -DCMAKE_CXX_FLAGS="/WX" -DCMAKE_TOOLCHAIN_FILE=c:/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_INSTALL_PREFIX="%PROJ_DIR%" + cmake -GNinja .. -DCMAKE_BUILD_TYPE="${{ env.BUILD_TYPE }}" -DBUILD_SHARED_LIBS="${{ env.BUILD_SHARED_LIBS }}" -DCMAKE_C_FLAGS="/WX" -DCMAKE_CXX_FLAGS="/WX" -DCMAKE_TOOLCHAIN_FILE=c:/vcpkg/scripts/buildsystems/vcpkg.cmake -DCMAKE_INSTALL_PREFIX="%PROJ_DIR%" ninja -v ninja install dir %PROJ_DIR%\bin @@ -55,10 +61,10 @@ jobs: set PROJ_DIR=%GITHUB_WORKSPACE%\proj_dir set PROJ_BUILD=%GITHUB_WORKSPACE%\build cd %PROJ_BUILD% - ctest -V -C Release + ctest -V -C "${{ env.BUILD_TYPE }}" set PATH=%PROJ_DIR%\bin;%PATH% - call %GITHUB_WORKSPACE%\test\postinstall\test_cmake.bat %PROJ_DIR% - proj + if "${{ env.BUILD_TYPE }}"=="Release" call %GITHUB_WORKSPACE%\test\postinstall\test_cmake.bat %PROJ_DIR% + if "${{ env.BUILD_TYPE }}"=="Release" proj MSYS2: runs-on: windows-latest diff --git a/src/iso19111/io.cpp b/src/iso19111/io.cpp index 5f2472a7..f8a4672a 100644 --- a/src/iso19111/io.cpp +++ b/src/iso19111/io.cpp @@ -7678,7 +7678,7 @@ const std::string &PROJStringFormatter::toString() const { iterCur = steps.erase(iterPrev, std::next(iterCur)); if (iterCur != steps.begin()) iterCur = std::prev(iterCur); - if (iterCur == steps.begin()) + if (iterCur == steps.begin() && iterCur != steps.end()) ++iterCur; }; |