The bug was logical: first the check was done for one of the values to be non-nil, and after that one of the values was assumed to be non-nil, while it could have been nil.
While fixing that, linter figured out that raw metadata config is never needed outside of acquireConfig, so this got dropped as well, simplifying the code even more.
smira
commented
4 weeks ago
The bug was logical: first the check was done for one of the values to be non-nil, and after that one of the values was assumed to be non-nil, while it could have been nil.
While fixing that, linter figured out that raw metadata config is never needed outside of
acquireConfig, so this got dropped as well, simplifying the code even more.Fixes #9578