Re: [PATCH] core: extend config objects

[Bartosz Golaszewski <brgl@xxxxxxxx>]

On Fri, Aug 6, 2021 at 3:27 PM Bartosz Golaszewski <brgl@xxxxxxxx> wrote:
>
> Kent suggested that we may want to add getters for the config objects in
> his reviews under the C++ patches. Indeed when working on Python bindings
> I noticed it would be useful for implementing __str__ and __repr__
> callbacks. In C++ too we could use them for overloading stream operators.
>
> This extends the config objects with getters. They are straightforward for
> the request config but for the line config, they allow to only read
> per-offset values that would be used if the object was used in a request
> at this moment. We also add getters for the output values: both taking
> the line offset as argument as well as ones that take the index and allow
> to iterate over all configured output values.
>
> The sanitization of input for the getters has subsequently been changed
> so that we never return invalid values. The input values are verified
> immediately and if an invalid value is passed, it's silently replaced
> by the default value for given setting.
>
> This patch also adds the reset function for the line config object - it
> can be used to reset all stored configuration if - for example - the
> config has become too complex.
>
> As this patch will be squashed into the big v2 patch anyway, I allowed
> myself to include some additional changes: some variable renames and
> other minor tweaks.
>
> Signed-off-by: Bartosz Golaszewski <brgl@xxxxxxxx>
> ---

Please disregard this one, I forgot to use the correct prefix. Sorry
for the noise.

Bart