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

Re: [PATCH v2 01/17] strbuf: make strbuf_getline_crlf() global

From: Johannes Schindelin <hidden>
Date: 2016-06-15 23:07:37

Possibly related (same subject, not in this thread)

Hi Junio,

On Wed, 16 Dec 2015, Junio C Hamano wrote:
Often we read "text" files that are supplied by the end user
(e.g. commit log message that was edited with $GIT_EDITOR upon
'git commit -e'), and in some environments lines in a text file
are terminated with CRLF.  Existing strbuf_getline() knows to read
a single line and then strip the terminating byte from the result,
but it is handy to have a version that is more tailored for a "text"
input that takes both '\n' and '\r\n' as line terminator (aka
<newline> in POSIX lingo) and returns the body of the line after
stripping <newline>.

Recently reimplemented "git am" uses such a function implemented
privately; move it to strbuf.[ch] and make it available for others.
... While at it, we fix the formatting of the strbuf_getline() function.

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