Home | About | Sematext search-lucene.com search-hadoop.com
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hadoop >> mail # dev >> trailing whitespace


Copy link to this message
-
Re: trailing whitespace
OK, if folks want to do something to get rid of trailing whitespace in the
project I won't object, but it doesn't seem like that big a deal to me. A
pre-commit hook makes sense to me. I just don't want to see the QA bot flag
patches containing trailing whitespace, thus requiring more round trips on
patches.

--
Aaron T. Myers
Software Engineer, Cloudera

On Mon, Nov 26, 2012 at 10:53 AM, Radim Kolar <[EMAIL PROTECTED]> wrote:

>  I've never understood why folks get worked up over a little trailing
>> whitespace here and there, since you can't see it and it doesn't affect
>> correctness. Spurious whitespace changes that make a review harder - those
>> are annoying. Trailing whitespace inadvertently left on lines where
>> legitimate changes were made in a patch - doesn't seem too harmful to me.
>>
>
> Trailing whitespace is annoying because:
>     if you have editor to set killing it, it will produce large patch.
>     if use use scroll up at end of line, then cursor will not jump to end
> of text but some space after it, it cost you more clicks for cursor
> movement and it is annoying if it ends of split line.
>     its good and standard practise to avoid it, git and other tools
> highlight it in red.
>     if you use ignore whitespace in git diff, it often produces patch
> failing to apply
>
> Trailing whitespace can be striped by pre-commit hook.
>
NEW: Monitor These Apps!
elasticsearch, apache solr, apache hbase, hadoop, redis, casssandra, amazon cloudwatch, mysql, memcached, apache kafka, apache zookeeper, apache storm, ubuntu, centOS, red hat, debian, puppet labs, java, senseiDB