Great! I’ll get it merged into the main branch and then I need to take care of a few repo things before moving on to the cpplint log. The cppcheck log is vaguer so I’ll start with the former to whittle it down 🙂
Regarding the 100 character line length: I feel the old standard of 80 is archaic. It goes back to the days of 800×600 resolution monitors. These days (in fact, beginning around 2010 or so), there are much fewer of those around so it’s time to bump up the line length. Why 100? It also lets you shrink the window to look at other things such as documentation. It doesn’t need the window maximized to see it in full but still looks better on today’s better screens.