fix(Exception): allow to return custom exceptions

[BotellaA]

No description provided.

[github-actions]

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v20.1.8) reports: 8 concern(s)

• include/geode/basic/library.hpp:130:9: warning: [cppcoreguidelines-macro-usage]

  variadic macro 'check_assertion' used; consider using a 'constexpr' variadic template function

    130 | #define check_assertion( condition, ... )                                      \
        |         ^

• include/geode/basic/library.hpp:130:9: warning: [readability-identifier-naming]

  invalid case style for macro definition 'check_assertion'

    130 | #define check_assertion( condition, ... )                                      \
        |         ^~~~~~~~~~~~~~~
        |         CHECK_ASSERTION

• include/geode/basic/library.hpp:139:9: warning: [cppcoreguidelines-macro-usage]

  variadic macro 'check_exception' used; consider using a 'constexpr' variadic template function

    139 | #define check_exception( condition, data, type, ... )                          \
        |         ^

• include/geode/basic/library.hpp:139:9: warning: [readability-identifier-naming]

  invalid case style for macro definition 'check_exception'

    139 | #define check_exception( condition, data, type, ... )                          \
        |         ^~~~~~~~~~~~~~~
        |         CHECK_EXCEPTION

• include/geode/basic/library.hpp:158:5: warning: [concurrency-mt-unsafe]

  function is not thread safe

    158 |     OPENGEODE_LIBRARY( opengeode_basic_api, OpenGeode, Basic );
        |     ^
  /__w/OpenGeode/OpenGeode/include/geode/basic/library.hpp:114:21: note: expanded from macro 'OPENGEODE_LIBRARY'
    114 |                     exit( 1 );                                                 \
        |                     ^

• include/geode/basic/library.hpp:158:5: warning: [cppcoreguidelines-missing-std-forward]

  forwarding reference parameter 'condition' is never forwarded inside the function body

    158 |     OPENGEODE_LIBRARY( opengeode_basic_api, OpenGeode, Basic );
        |     ^
  /__w/OpenGeode/OpenGeode/include/geode/basic/library.hpp:104:25: note: expanded from macro 'OPENGEODE_LIBRARY'
    104 |             Condition&& condition, MessageProvider&& message_provider )        \
        |                         ^

• include/geode/basic/library.hpp:158:5: warning: [cppcoreguidelines-missing-std-forward]

  forwarding reference parameter 'data_provider' is never forwarded inside the function body

    158 |     OPENGEODE_LIBRARY( opengeode_basic_api, OpenGeode, Basic );
        |     ^
  /__w/OpenGeode/OpenGeode/include/geode/basic/library.hpp:91:28: note: expanded from macro 'OPENGEODE_LIBRARY'
     91 |             DataProvider&& data_provider,                                      \
        |                            ^

• include/geode/basic/library.hpp:158:5: warning: [cppcoreguidelines-missing-std-forward]

  forwarding reference parameter 'message_provider' is never forwarded inside the function body

    158 |     OPENGEODE_LIBRARY( opengeode_basic_api, OpenGeode, Basic );
        |     ^
  /__w/OpenGeode/OpenGeode/include/geode/basic/library.hpp:93:31: note: expanded from macro 'OPENGEODE_LIBRARY'
     93 |             MessageProvider&& message_provider )                               \
        |                               ^

Have any feedback or feature suggestions? Share it here.

[BotellaA]

🎉 This PR is included in version 17.4.4-rc.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

[BotellaA]

🎉 This PR is included in version 17.4.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀