I'm sorry that I had forgotten to review the formatting first, but to be fair this block was just added. I would have caught the blank line otherwise in the first review. When I +1'ed the patch I only reviewed behavioral changes because I had mistakenly assumed you are using erlang.el and the file local vars would do the rest. Also, just because there are superfluous blank lines doesn't mean we should add more. It took considerable effort to get the code base into a consistent formatting state. Therefore, I try to prevent introducing inconsistency. While the superfluous blank line works for me and doesn't prevent merging this, the merge is up to Dizzy as only he writes to rebar/rebar.git. So, what can be fixed before Dizzy gets to review this helps speed up his review+merge process. The blank line can, but doesn't have to be omitted :-).