Well, instead it should just be a local variable I think.  That is:

```c++
// TODO: Be smart about same language, different country
auto langValuesMapping = GetLangValuesMapping();
if (!defaultLangRegion.empty()) {
```

It doesn't need to live longer than the `Config::Load()` function, so it doesn't need to be a member variable.  Member variables are only needed for things that are used outside a single function.

-[Unknown]