Pull request for all the remaining patches, including the new performance monitors.

On Fri, 2014-10-10 at 04:47 -0700, Jens Breitbart wrote:
> Inline: depending on the concrete use the compiler may remove the copy
> of the byte array, so it is not about potential function call
> overhead. My guess it that the arrays are small atm, so there is not
> much to gain as it is.

Good point. I have modified the function to take a const reference now,
that should remove the need for unnecessary copies.

> Nonetheless, I would still remove it from the class header file all
> together, as it seems not to be specific to the Clustsafe. You may
> just put it in the .cpp file

Unfortunately, that would mean, that the calculateChecksum() function
would need to defined (and thereby declared) before ever being used (C/C
++ are really showing their age in that regard). That's currently the
case, but things change, so I'd rather not risk it.

>  or move it to the tools.cpp file.

In theory, that would be the right place for such a function. However, I
really don't want anybody to think, that I consider this thing a valid
checksum algorithm. This kind of code should never ever be used by
anybody else.