> I don't follow this rule.
Do you mind if the patches move member functions to .cpp files? As I see it, headers shouldn't include implementation details.
> One person's extra whitespace is another persons useful code separation.
What I meant by stray whitespace is that there are lines that have 3 or 4 spaces at the end where they serve no purpose. It's the kind of whitespace that Eclipse's "Remove trailing whitespace" removes and the kind that Checkstyle warns you about.
I'll update the Linux build files in a new patch.
Meanwhile, here's a signedness fix from StompFrame.h.