[Freeswitch-dev] trailing white spaces in code cleanup suggestions and jira question

Anthony Minessale anthony.minessale at gmail.com
Mon Apr 9 19:35:27 MSD 2012


Isn't it only a problem if you clean your working code differently
from the remote?
You should resist doing any local whitespace changes and we can make a
tradition to maybe clean up the tree at every major release in a
single sweep so the history is not tainted by tons of whitespace
changes.

On Mon, Apr 9, 2012 at 7:50 AM, Rafael Vargas <vargas.khomp at gmail.com> wrote:
> My two cents:
>
> As already stated by Ken Rice, trailing spaces are not a problem. And I
> personally had a bad experience in purely aesthetic code cleanups. It messes
> up the "blame" function of the VCS (git in this case), once the last
> modifications in the the altered lines will be assigned to whoever commited
> the cleanup.
>
> --
> Rafael Vargas
>
>
> On Mon, Apr 9, 2012 at 09:24, Seven Du <dujinfang at gmail.com> wrote:
>>
>> I also prefer small, controlled updates, however I can't do git push and
>> jira would be a little pain ...
>> Ken mentioned there's nothing wrong with white spaces in another mail, I
>> agree but still think it's a little pain to generate "clean" diff when
>> there's many trailing spaces involved. And as I said doing this could affect
>> all devs So I'd like to get their ideas, especially we have more branches
>> than before it's means need more merge and possible there's conflicts...
>>
>> 7.
>>
>> On Sunday, April 8, 2012 at 4:01 AM, Michael Collins wrote:
>>
>> /usr/src/freeswitch/src# grep -rc "\s\+$" * | grep -v ":0$"
>>
>> 1200+ files with trailing whitespace does seem a bit excessive, however I
>> don't know how large a "problem" it really is. If you're going to do
>> whitespace cleanup I recommend starting small, like with mod_rtmp.c. Run
>> your whitespace cleanup script on just a few files, recompile them, verify
>> that it's all good, then commit & push. I don't know about the other devs,
>> but I would feel much more comfortable with small, controlled updates than a
>> large sweep that does a lot of files.
>>
>> Just my $0.02...
>>
>> -MC
>>
>> On Sat, Apr 7, 2012 at 8:49 AM, Seven Du <dujinfang at gmail.com> wrote:
>>
>> cat -t mod_rtmp.c | grep I
>> cat mod_rtmp.c | grep " $"
>>
>> some editors insert tabs automatically when you press enter.
>>
>> On Saturday, April 7, 2012 at 12:26 PM, Michael Collins wrote:
>>
>> can you show us an example of code w/ trailing whitespaces?
>> -MC
>>
>>
>> _________________________________________________________________________
>> Professional FreeSWITCH Consulting Services:
>> consulting at freeswitch.org
>> http://www.freeswitchsolutions.com
>>
>> 
>> 
>>
>> Official FreeSWITCH Sites
>> http://www.freeswitch.org
>> http://wiki.freeswitch.org
>> http://www.cluecon.com
>>
>> FreeSWITCH-dev mailing list
>> FreeSWITCH-dev at lists.freeswitch.org
>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
>> http://www.freeswitch.org
>>
>>
>>
>> _________________________________________________________________________
>> Professional FreeSWITCH Consulting Services:
>> consulting at freeswitch.org
>> http://www.freeswitchsolutions.com
>>
>> 
>> 
>>
>> Official FreeSWITCH Sites
>> http://www.freeswitch.org
>> http://wiki.freeswitch.org
>> http://www.cluecon.com
>>
>> FreeSWITCH-dev mailing list
>> FreeSWITCH-dev at lists.freeswitch.org
>> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
>> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
>> http://www.freeswitch.org
>>
>
>
> _________________________________________________________________________
> Professional FreeSWITCH Consulting Services:
> consulting at freeswitch.org
> http://www.freeswitchsolutions.com
>
> 
> 
>
> Official FreeSWITCH Sites
> http://www.freeswitch.org
> http://wiki.freeswitch.org
> http://www.cluecon.com
>
> FreeSWITCH-dev mailing list
> FreeSWITCH-dev at lists.freeswitch.org
> http://lists.freeswitch.org/mailman/listinfo/freeswitch-dev
> UNSUBSCRIBE:http://lists.freeswitch.org/mailman/options/freeswitch-dev
> http://www.freeswitch.org
>



-- 
Anthony Minessale II

FreeSWITCH http://www.freeswitch.org/
ClueCon http://www.cluecon.com/
Twitter: http://twitter.com/FreeSWITCH_wire

AIM: anthm
MSN:anthony_minessale at hotmail.com
GTALK/JABBER/PAYPAL:anthony.minessale at gmail.com
IRC: irc.freenode.net #freeswitch

FreeSWITCH Developer Conference
sip:888 at conference.freeswitch.org
googletalk:conf+888 at conference.freeswitch.org
pstn:+19193869900



Join us at ClueCon 2011 Aug 9-11, 2011
More information about the FreeSWITCH-dev mailing list