Stripping ALL tabs from the code

Bron Gondwana brong at fastmail.fm
Tue Jun 9 06:42:03 EDT 2015


The big news from yesterday's meeting is that we are going to check in a single massive commit which strips all leading tabs from the code, everywhere.  The new coding standard is 4 spaces for each level of indent, and spaces for all alignment within code as well.

I've pushed a small perl script to master which performs the necessary surgery.  it has the smarts to know when a tab isn't on a multiple-of-8 boundary and substitute fewer spaces.

The code still builds and passes tests afterwards, so I'm pretty sure it's good.

I will be running the on master on June 14th (Sunday) night Melbourne time.  That's weekend for everyone, everywhere in the world.  I will immediately push the result back to master (after running the tests again of course).

If you have a patch series lying around that you want to apply, it might be a good idea to do it before then!  Alternatively, you can patch with -l or --ignore-whitespace to apply your patches to the tree, and then run tools/remove-tabs.pl on the resulting tree to remove the tabs again.

Enjoy your new, simpler coding style with no trailing whitespace left!




-- 
  Bron Gondwana
  brong at fastmail.fm


More information about the Cyrus-devel mailing list