[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kstars-devel
Subject:    [Kstars-devel] whitespace: worth fixing?
From:       Jason Harris <kstars () 30doradus ! org>
Date:       2007-09-21 19:54:15
Message-ID: 200709211254.15150.kstars () 30doradus ! org
[Download RAW message or body]

Hello,

Whitespace is currently a bit of a mess in our codebase.  I've always tried to 
use TAB characters in my code, which I now kind of regret because many 
non-Jason people use spaces instead, and spaces are recommended by the 
kdelibs coding style guidelines[1].

We can fix the code in an automated was using astyle[2], but this will add a 
"useless" entry to the svn history of all files, and would require a large 
file transfer the next time you svn up.

Is it worth fixing the whitespace in our codebase to use 4 spaces instead of a 
TAB?  If so, does it make sense to do it now, before 4.0.0 is released, or 
should we wait until after the freeze is lifted for such a pervasive (but 
non-functional) change?

thanks,
Jason

[1]: http://techbase.kde.org/Policies/Kdelibs_Coding_Style#Indentation
[2]: http://astyle.sourceforge.net/

-- 
KStars: KDE Desktop Planetarium
kstars@30doradus.org
_______________________________________________
Kstars-devel mailing list
Kstars-devel@kde.org
https://mail.kde.org/mailman/listinfo/kstars-devel
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic