Thread (11 messages) flat view 11 messages, 5 authors, 2016-06-15

Re: [PATCH] pre-commit hook should ignore carriage returns at EOL

From: Christian Holtje <hidden>
Date: 2016-06-15 22:44:49

On Jun 24, 2008, at 2:26 PM, Ian Hilt wrote:
Perhaps you want to use printf "foo\r" instead?  echo "foo\r" does not
produce a CR on my system.
...
Wouldn't it be less redundant to do a test for \s\r$ here instead of
testing for \r$ and then \s\r$?

The code is checking for \r$ and then doing a different space check  
depending on that, not one after another.

Thanks for the feedback. I'll put up v2 in a second.

Ciao!
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help