indentations and deletion of commented out lines only. should be a quick review...

Hey cloomis: this came up at today's meeting. Do you have an ETA on a review?

Setting back to cloomis for review.

If this really is just an indentation cleanup, can I suggest re-starting from a merged HEAD? Especially if you are just using a program to reformat. The number of hard-to-judge "diff changes" in the first commit on PIPE2D-88 (46acb17) makes it hard for me to believe that it would merge to master cleanly.

As an aside, I should look into how to adjust diff options to handle this kind of changeset.

Much better, thanks. This is definitely cleanly reviewable and mergeable. Functionally, I check off on this.

But: if you have a tool you can rerun with args, can you run with a line-length limit of 110 chars? There are quite a few long lines left, some exceeding 250 characters.
That said, most of those are in very long cout << lines. If those are to be replaced with a logger ticket it might not be worth it.

Another note on the same subject: your formatter did insert indents for the namespaces:

#include "pfs/drp/stella/math/Math.h"
namespace pfs {
  namespace drp {
    namespace stella {
      namespace math {

        template< typename T, typename U >
        [ ... ]

If there is an option to disable that you would lessen the visual price.

I removed the line indent for the namespaces and filed a ticket to reduce the line length to the permitted 110 characters. Can I merge?

Yes. Please do.

Merged into master