While reviewing all the new code that's been added I'm finding it difficult to spot where one namespace-scope definition ends and another starts because the spacing between them (the number of newlines) is the same as the spacing between members, namely 1 blank line. I find code easier to read when namespace scope definitions of functions and classes that span more than one line are separated by two blank lines.
Existing code likely isn't completely consistent in this regard, and I'm sure examples of both styles could be found, but I'd like to think the two-line style is prevalent. Either way, in the interest of readability, I'd like to suggest that we adopt the two-line spacing style for all new code. Yes? Martin
