GoodRunsLists: thread-safety fixes and code cleanup

The main change is to remove the thread-unsafe caching of the previous
run and LB number in `TGoodRunsList::HasRun[LumiBlock]`. If the
`std::map` lookup is really too slow we need to implement this properly.

Also modernize the code and enable the thread-checker.