On ma, 2016-08-29 at 10:06 +0200, Johannes Schindelin wrote:
> + if (strbuf_read_file(buf, path, 0) < 0) {
> + warning_errno("could not read '%s'", path);
> + return 0;
> + }
> +
> + if (buf->len > orig_len && buf->buf[buf->len - 1] == '\n') {
> + if (--buf->len > orig_len && buf->buf[buf->len - 1]
> == '\r')
> + --buf->len;
> + buf->buf[buf->len] = '\0';
> + }
Why not use open + strbuf_getline instead of hand-rolling a newline
eradicator?
D.