search

EVerest / everest-framework

Apache License 2.0
21 stars 19 forks source link

Fix missing Exception throw on loading config #123

Closed andistorm closed 1 year ago

andistorm commented 1 year ago

In case of loading a broken config.yaml the function load_yaml returns a nullptr. This fix avoid accessing non-existing elements on a nullptr.

hikinggrass commented 1 year ago

Could we move this check into load_yaml itself? @a-w50 we're using load_yaml in quite a few places in the code and I don't see any usage that would rely on returning a nullptr in case of a broken/empty yaml here. Or am I missing something?