container: root/plugins: factor out internal types for common use, rewrite throw handler, fix plugin example's usage of throw handler #244

2025-11-21T04:21:30Z

AaronFiore commented

2025-11-21 04:21:30 +00:00

(Migrated from gitea.com)

Description

These types and handlers are too generic to be kept internal only.

They can now be used across all root macros, plugins and overall API.

Refactor using common types that were once internal
Removes ancient macro approaches to C++20 solutions
Changes Exception message type to use std::string
- std::string_view isn't worthwhile in this context

The old macros should not have been used in any example; as they
were intended for internal use only.

This was noted in Doxygen ...by not noting them as publicly consumable.

NOTE: adds deprecation warnings to formerly internal types that are now aliased internally.

Agreement

I have read and understood the Development and License documentation

### Description These types and handlers are too generic to be kept internal only. They can now be used across all root macros, plugins and overall API. --- - Refactor using common types that were once internal - Removes ancient macro approaches to C++20 solutions - Changes `Exception` message type to use std::string * std::string_view isn't worthwhile in this context --- The old macros should *not* have been used in any example; as they were intended for internal use only. This was noted in Doxygen ...by not noting them as publicly consumable. --- NOTE: adds deprecation warnings to formerly internal types that are now aliased internally. ### Agreement - [x] I have read and understood the [Development](https://gitea.com/EvergreenCrypto/docker-finance#development) and [License](https://gitea.com/EvergreenCrypto/docker-finance#license-and-disclaimer) documentation

AaronFiore commented

2025-11-21 19:20:17 +00:00

(Migrated from gitea.com)

All tests pass.

I'm still not convinced that more internal types couldn't be publicly consumable (especially after more c++20 refactoring). TBD.

All tests pass. I'm still not convinced that more internal types couldn't be publicly consumable (especially after more c++20 refactoring). TBD.

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Reference: EvergreenCrypto/docker-finance#244