From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 23 Sep 2011 19:04:48 GMT
Subject: Re: [PATCH 3/6] branch: teach --edit-description option
Message-ID: <7v62kjulkf.fsf@alter.siamese.dyndns.org>
In-Reply-To: <20110923094721.GA8397@duynguyen-vnpc>

Nguyen Thai Ngoc Duy <pclouds@gmail.com> writes:

> On Thu, Sep 22, 2011 at 03:09:19PM -0700, Junio C Hamano wrote:
>> +	if (launch_editor(git_path(edit_description), &buf, NULL)) {
>> +		strbuf_release(&buf);
>> +		return -1;
>> +	}
>> +	stripspace(&buf, 1);
>> +
>> +	strbuf_addf(&name, "branch.%s.description", branch_name);
>> +	status = git_config_set(name.buf, buf.buf);
>
> I suppose a Windows editor mave save the description with \r\n
> ending. Perhaps a patch like this to avoid messing up config file?

Doesn't stripspace() cleanse that already?