Skip to content
CI

Fix posix ApplyEnvironmentModifications for repeated modifications #2280

Sign in to view logs

Fix posix ApplyEnvironmentModifications for repeated modifications

Fix posix ApplyEnvironmentModifications for repeated modifications #2280

Triggered via pull request February 27, 2024 22:29
@quittlequittle
opened #3260
v1.x
Status Failure
Total duration 1m 14s
Artifacts –

ci.yml

on: pull_request
Commit message check
6s
Commit message check
Code formatting check
1m 1s
Code formatting check
Baseline cmake check
0s
Baseline cmake check
Documentation Build
0s
Documentation Build
Android
0s
Android
Matrix: Linux
Matrix: Mac
Matrix: Windows
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 2 warnings
Code formatting check
clang-format issues were found. See CONTRIBUTING.md for more information. diff --git a/renderdoc/os/posix/posix_process.cpp b/renderdoc/os/posix/posix_process.cpp index c2d1f3f..21716d8 100644 --- a/renderdoc/os/posix/posix_process.cpp +++ b/renderdoc/os/posix/posix_process.cpp @@ -415,7 +415,7 @@ void ApplyEnvironmentModifications(rdcarray<EnvironmentModification> &modificati { EnvironmentModification &m = modifications[i]; - rdcstr& value = currentEnv[m.name.c_str()]; + rdcstr &value = currentEnv[m.name.c_str()]; ApplySingleEnvMod(m, value);
Code formatting check
Process completed with exit code 1.
Commit message check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Code formatting check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.