Oh, good thought. Sounds like the best time for this would be during Hard Freeze, when no one but release managers can commit anyway, but we've not yet branched. I'll stick it in there unless anyone feels strongly otherwise.
Bryce
On Sat, Nov 03, 2007 at 10:53:55PM +0100, J.B.C.Engelen@...1578... wrote:
See: http://wiki.inkscape.org/wiki/index.php/InkscapeJanitors#Cleanup:_Whites pace
I vote for mass cleansing of tabs and trailing spaces at the start of feature freeze for this release. I'm afraid it is never going to happen otherwise :/
Johan
-----Original Message----- From: inkscape-devel-bounces@lists.sourceforge.net [mailto:inkscape-devel-bounces@lists.sourceforge.net] On Behalf Of Diederik van Lierop Sent: zaterdag 3 november 2007 22:34 To: inkscape-devel@lists.sourceforge.net Subject: [Inkscape-devel] tabs
Hi,
After my commit of today, Johan Engelen pointed out that there were quite some tabstops being used in it instead of spaces. I started an automated find & replace, replacing all tabs with four spaces in just that commit. Worked like a charm.
I also found tens of thousands of tabs when searching just *.h in /src. Now I wonder: should we do a massive search & replace on the full code base? I've tried and it compiles nicely. It also doesn't mess up the indenting, at least not in the places I've looked at. Is it worth it? Or will it give to much troubles in the end afterall?
Diederik
This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Inkscape-devel mailing list Inkscape-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/inkscape-devel