Re: [RFC PATCH v3 5/9] Use new functions in git_rebase__interactive
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Mar 23, 2018, 18:24 UTC
- Message-ID
- <xmqqzi2yiraj.fsf@gitster-ct.c.googlers.com>
- In-Reply-To
- <nycvar.QRO.7.76.6.1803231838020.77@ZVAVAG-6OXH6DA.rhebcr.pbec.zvpebfbsg.pbz>
Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
Show 28 quoted lines
> If you fold this into the previous patch, I am sure that after your 3/9
> indenting the function properly, the splitting into functions will look
> more or less like this:
>
> -git_rebase__interactive () {
> +initiate_action () {
> + action="$1"
>
> [... unchanged code ...]
> +}
> +
> +<next function> () {
> + [... setting up variables ...]
>
> [.. unchanged code ...]
> +}
> [... more of the same ...]
> +
> +git_rebase__interactive () {
> + initiate_action "$action" &&
> + <next function> <arguments> &&
> + ...
> +}
>
> In other words, it would be easier to review and to verify that the
> previous code is left unchanged (although that would have to be verified
> manually by applying 3/9 and then looking at the diff with the -w option,
> anyway).We are on the same page on this. A series structured to have a step that looks exactly like the above would be very pleasant to review.
Thanks for a great suggestion.