Re: [PATCH v3] builtin/revert.c: refactor using an enum for cmd-action
- From
Junio C Hamano <gitster@pobox.com>
- Date
- Jan 12, 2024, 00:40 UTC
- Message-ID
- <xmqqttnjcrk6.fsf@gitster.g>
- In-Reply-To
- <20240111200627.64199-1-mi.al.lohmann@gmail.com>
Michael Lohmann <mi.al.lohmann@gmail.com> writes:
Show 14 quoted lines
> - if (cmd == 'c') > + case ACTION_CONTINUE: > return sequencer_continue(the_repository, opts); > - if (cmd == 'a') > + case ACTION_ABORT: > return sequencer_rollback(the_repository, opts); > - if (cmd == 's') > + case ACTION_SKIP: > return sequencer_skip(the_repository, opts); > - return sequencer_pick_revisions(the_repository, opts); > + case ACTION_START: > + return sequencer_pick_revisions(the_repository, opts); > + } > }
This change broke the build when merged to 'seen' like so ...
builtin/revert.c: In function 'run_sequencer':
builtin/revert.c:242:1: error: control reaches end of non-void function [-Werror=return-typ ]
242 | }
| ^
... so I'm discarding
it out of my tree and redoing today's integration cycle.Different compilers are smart in different ways, and we shouldn't overly rely on the fact that some compilers may be happy by seeing a switch that has all the enum values and notice that one of the return will be triggered in its case arms.