Boost C++ Libraries: Ticket #10385: init_from_stream ignores errors in config file/stream

status changed; resolution set

Andrey Semashev — Wed, 20 Aug 2014 13:57:56 GMT

status new → closed
resolution → wontfix

Technically, this is a correct config with section Sink and a nested section ImportantSink. The library simply doesn't use that section, but that doesn't make the config incorrect. I don't think that issuing errors upon any unrecognized sections or parameters is a good idea because they can potentially be used by the user.

Antony Polukhin — Wed, 20 Aug 2014 14:12:46 GMT

How about providing an option for forcing strict/non-strict config file parsing, like it is done in Boost.ProgramOptions.

Having it set so "strict" by default will help beginners to find errors in their config files, while advanced users still will be able to put all the configs in one file.

Andrey Semashev — Wed, 20 Aug 2014 15:47:42 GMT

Having an optional strict mode might be useful, although it may affect public interfaces for extending the library. I'll have to think about it. In any case, I don't think it should be the default because the current behavior is different.