Skip to content

Conversation

maloel
Copy link
Contributor

@maloel maloel commented Jul 29, 2024

PR #13192 caused warnings under Linux:

In file included from /home/jenkins/workspace/LRS_linux_compile_lib_ci/src/core/option-interface.h:8,
                 from /home/jenkins/workspace/LRS_linux_compile_lib_ci/src/option.h:5,
                 from /home/jenkins/workspace/LRS_linux_compile_lib_ci/src/source.cpp:6:
/home/jenkins/workspace/LRS_linux_compile_lib_ci/third-party/rsutils/include/rsutils/json-fwd.h:41:17: warning: inline function 'bool rsutils::json_base::exists() const' used but never defined
   41 |     inline bool exists() const;
      |                 ^~~~~~

Moved inline operator bool() to json.h.

Fixed additional Python SyntaxWarning: invalid escape sequence '\ ' warnings for backslash usage with invalid following character.

@maloel maloel requested a review from OhadMeir July 29, 2024 05:41
@maloel maloel merged commit b0d4926 into IntelRealSense:development Jul 29, 2024
22 checks passed
@maloel maloel deleted the json-warnings branch July 29, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants