Re: [PATCH v2 1/2] branch: refactor "edit_description" code path

From: Rubén Justo <rjusto@gmail.com>
Date: Sep 7, 2022, 21:24 UTC
Message-ID: <a5e5f94d-965b-3766-60c2-27183148e052@gmail.com>
In-Reply-To: <xmqq7d2fszhk.fsf@gitster.g>

On 9/7/22 10:25 PM, Junio C Hamano wrote:

Maybe the return 1 when edit_branch_description(), was a typo I was maintaining?

Show 9 quoted lines

>> -		}
>> -		strbuf_release(&branch_ref);
>> +		} else
>> +			ret = edit_branch_description(branch_name);
>>  
>> -		if (edit_branch_description(branch_name))
>> -			return 1;
>> +		strbuf_release(&branch_ref);
>> +		return ret;

Thanks for you review.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.