@bocchino @thomas-bc Following up on the tagup discussion relating to this PR:

I tried to change the autocoder to do const char* severityString = "SEVERITY ERROR"; and then remove assignment in the default case. Interestingly this gives the same error (seems like a false positive?):

Value stored to 'severityString' during its initialization is never read

So I think we have a couple options to keep the static analyzer happy:

1. const char* severityString; as currently proposed and keep the switch case as-is
2. const char* severityString = nullptr; and keep the switch case as-is
3. const char* severityString = "UNKNOWN"; //NOLINT(clang-analyzer-deadcode.DeadStores) and keep the switch case as-is
4. const char* severityString = "SEVERITY ERROR"; //NOLINT(clang-analyzer-deadcode.DeadStores) and update the default switch case to be empty/just break

Preferences?

That will give the same error.

So to initialize it without warning I think we'd need to use the //NOLINT (and might as well keep the UNKNOWN in that case, I think)