Re: [PATCH v2] allow "-" as short-hand for "@{-1}" in "branch -d"

From: Junio C Hamano <gitster@pobox.com>
Date: Aug 16, 2022, 18:55 UTC
Message-ID: <xmqqo7wkt3w7.fsf@gitster.g>
In-Reply-To: <pull.1315.v2.git.1660669912043.gitgitgadget@gmail.com>

"Rubén Justo via GitGitGadget"  <gitgitgadget@gmail.com> writes:

> From: rjusto <rjusto@gmail.com>

Documentation/SubmittingPatches:

    Also notice that a real name is used in the `Signed-off-by`
    trailer. Please don't hide your real name.

>      -    branch: allow "-" as a short-hand for "previous branch"
>      +    allow "-" as short-hand for "@{-1}" in "branch -d"

The "branch:" prefix is lost here, which is not an improvement. The "<area>:" prefix is what makes it easier to locate a particular change in "git shortlog --no-merges v2.37.0..v2.38.0".

As to the implementation, there is nothing to complain about, but as we already discussed during the reivew of the first iteration, I am not sure if the goal is sound in the first place.

Thanks.