Skip to content

Commit

Permalink
[OMON-736] Enable C++20 and fix related warnings
Browse files Browse the repository at this point in the history
  • Loading branch information
awegrzyn committed Nov 21, 2023
1 parent d3007d1 commit 822e484
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ endif()

# Define project
project(Monitoring
VERSION 3.17.6
VERSION 3.18.0
DESCRIPTION "O2 Monitoring library"
LANGUAGES CXX
)
Expand Down Expand Up @@ -217,7 +217,7 @@ target_compile_definitions(Monitoring
)

# Use C++17
target_compile_features(Monitoring PUBLIC cxx_std_17)
target_compile_features(Monitoring PUBLIC cxx_std_20)


####################################
Expand Down
2 changes: 1 addition & 1 deletion src/ProcessDetails.cxx
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ inline void ProcessDetails::generateProcessName()
#endif

if (mProcessName.empty()) {
mProcessName = "!";
mProcessName = "<undefined>";

Check warning on line 62 in src/ProcessDetails.cxx

View check run for this annotation

Codecov / codecov/patch

src/ProcessDetails.cxx#L62

Added line #L62 was not covered by tests
} else {
mProcessName = mProcessName.substr(mProcessName.find_last_of("/") + 1);
}
Expand Down

0 comments on commit 822e484

Please sign in to comment.