Fine with me too. I had already approved in !45.

This looks fine.

I'm wondering whether it would make sense to define enum for individual variables to use instead of strings for names. Then we could split this huge function to set of functions (in namespace or class) for single variable. The generic getValue function could then have array of pointers to functions (or switch-case) to call right one based on the variable name.

This was indeed closed by D136 (see also !44).