First of all, great library; it allows us to simplify our config parsing significantly.
I have one change request: currently when you set rfl::NoExtraFields it will warn on unused fields.
This works correctly, but the error message contains the following string:
Remove the rfl::NoExtraFields processor or add rfl::ExtraFields to avoid this error message.
I can see how this is useful for developers, but we want to display these error message to the users, so that they know exactly why parsing failed. This part of the message is not exactly useful though, because a user can't change any of these internals.
Could this part of the message be removed? I'd be happy to make the change myself.
For reference:
https://github.com/getml/reflect-cpp/blob/main/include/rfl/parsing/ViewReader.hpp#L145
First of all, great library; it allows us to simplify our config parsing significantly.
I have one change request: currently when you set
rfl::NoExtraFieldsit will warn on unused fields.This works correctly, but the error message contains the following string:
I can see how this is useful for developers, but we want to display these error message to the users, so that they know exactly why parsing failed. This part of the message is not exactly useful though, because a user can't change any of these internals.
Could this part of the message be removed? I'd be happy to make the change myself.
For reference:
https://github.com/getml/reflect-cpp/blob/main/include/rfl/parsing/ViewReader.hpp#L145