Hu Jialun <hujialun@xxxxxxxxxxxxxxx> writes: > Strings of hint messages inserted into editor on interactive commit was > scattered in-line, rendering the code harder to understand at first > glance. > > Extract those messages out into separate variables to make the code > outline easier to follow. > > Helped-by: Junio C Hamano <gitster@xxxxxxxxx> > Helped-by: Đoàn Trần Công Danh <congdanhqx@xxxxxxxxx> > Signed-off-by: Hu Jialun <hujialun@xxxxxxxxxxxxxxx> > --- > builtin/commit.c | 20 ++++++++++---------- > 1 file changed, 10 insertions(+), 10 deletions(-) > > diff --git a/builtin/commit.c b/builtin/commit.c > index 190d215d43..e68d139dee 100644 > --- a/builtin/commit.c > +++ b/builtin/commit.c > @@ -889,6 +889,14 @@ static int prepare_to_commit(const char *index_file, const char *prefix, > int ident_shown = 0; > int saved_color_setting; > struct ident_split ci, ai; > + const char *hint_cleanup_all = _("Please enter the commit message for your changes." > + " Lines starting\nwith '%c' will be ignored," > + " and an empty message aborts the commit.\n"); > + const char *hint_cleanup_space = _("Please enter the commit message for your changes." > + " Lines starting\n" > + "with '%c' will be kept; you may remove them" > + " yourself if you want to.\n" > + "An empty message aborts the commit.\n"); That would easily make lines that are overly long. Perhaps fold them like so? const char *hint_cleanup_all = _("Please enter the commit message for your changes." " Lines starting\nwith '%c' will be ignored," " and an empty message aborts the commit.\n");