Alban Gruin <alban.gruin@xxxxxxxxx> writes: > This makes rebase_path_todo(), get_missing_commit_check_level() and the > enum check_level accessible outside sequencer.c. check_level is renamed > missing_commit_check_level, and its value names are prefixed by > MISSING_COMMIT_ to avoid namespace pollution. It is not too big a deal, but we prefer to hear our story told in a consistent voice. We'd phrase often the above as if giving an order to the code base to "become like so", i.e. Make X, Y, and Z accessible outside sequencer.c; rename A to B and prefix its values by doing C to avoid namespace pollusion. > This function and this enum will eventually be moved to > rebase-interactive.c and become static again, so no special attention > was given to the naming. > > This will be needed for the rewrite of append_todo_help() from shell to > C, as it will be in a new library source file, rebase-interactive.c. > > Signed-off-by: Alban Gruin <alban.gruin@xxxxxxxxx> > --- > Unchanged from v2. You added an excuse for being sloppy in the naming to the log message; the log message is an important and integral part of the patch, so please do not label an update like this as unchanged. "eventually" is fine, but that means we cannot split the early parts of the topic, once they are polished well enough, into a separate bit and have that graduate earlier than the remainder of the topic until that "moved and become static again" happens, which may be a bit unfortunate. Let's make sure we can get to that "eventually" step soon.
