Explicitly use std::move

[WinterSolstice8]

I'm using this library in a project that makes heavy use of linters/clang format etc and not using qualified std::move was causing compilation errors specifically in clang.

I made this change to accommodate for that

[bombela]

The reason for using move unqualified is because it is std::move on C++11 and onward. But it is template <typename T> T &move(T &v) { return v; } on C++98. Which somewhat behaves a bit like std::move thanks to copy elision.

See what is being imported/set before/after C++11: https://github.com/bombela/backward-cpp/blob/master/backward.hpp#L414

[lahwaacz]

However, the build system requires C++11: https://github.com/bombela/backward-cpp/blob/master/CMakeLists.txt#L65

[bombela]

You do not have to use CMake. In fact I personally do not really maintain the CMake part. I barely know how it works to tell the truth. I leave it to contributors. The "build system" used to test backward-cpp is merely a shell script. Nothing more. All you need is the .hpp, set the defines used to configure which unwind and symbol resolution library you want to use. And then do not forget to link appropriately to the libraries you have chosen. And the documentation and list of libraries is found at the top of the .hpp. The .hpp must compile on C++98 and onward. I recommend using backward-cpp by simply copying the .hpp and adding whatever rule is necessary to your build system. Of course any contribution to the CMake configuration is welcome too :)

…

On Tue, Apr 15, 2025, 13:57 Jakub Klinkovský ***@***.***> wrote: However, the build system requires C++11: https://github.com/bombela/backward-cpp/blob/master/CMakeLists.txt#L65 — Reply to this email directly, view it on GitHub <#349 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABUZDCNXSHWJLVUFT3FG632ZVXCJAVCNFSM6AAAAABY2C3BUOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMBXGQ4DMOJRGM> . You are receiving this because you commented.Message ID: ***@***.***> *lahwaacz* left a comment (bombela/backward-cpp#349) <#349 (comment)> However, the build system requires C++11: https://github.com/bombela/backward-cpp/blob/master/CMakeLists.txt#L65 — Reply to this email directly, view it on GitHub <#349 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AABUZDCNXSHWJLVUFT3FG632ZVXCJAVCNFSM6AAAAABY2C3BUOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDQMBXGQ4DMOJRGM> . You are receiving this because you commented.Message ID: ***@***.***>