{"thread":{"id":"60387","subject":"[PATCH v2 1/1] [OUTREACHY] builtin/add.c: clean up die() messages","startedAt":"2023-10-17T22:13:56Z","lastAt":"2023-10-17T22:16:13Z","messageCount":2,"participants":["Naomi Ibe","Junio C Hamano"],"isPatch":true,"patchVersion":2,"patchTotal":1},"messages":[{"id":"483373","messageId":"20231017221323.352-1-naomi.ibeh69@gmail.com","threadId":"60387","inReplyTo":null,"subject":"[PATCH v2 1/1] [OUTREACHY] builtin/add.c: clean up die() messages","fromName":"Naomi Ibe","fromEmail":"naomi.ibeh69@gmail.com","sentAt":"2023-10-17T22:13:23Z","receivedAt":"2023-10-17T22:13:56Z","isPatch":true,"sender":{"key":"naomi.ibeh69@gmail.com","avatar":"https://avatars.githubusercontent.com/u/104323769?v=4"},"body":"As described in the CodingGuidelines document, a single line\nmessage given to die() and its friends should not capitalize its\nfirst word, and should not add full-stop at the end.\n\nSigned-off-by: Naomi Ibe <naomi.ibeh69@gmail.com>\n---\n builtin/add.c | 10 +++++-----\n 1 file changed, 5 insertions(+), 5 deletions(-)\n\ndiff --git a/builtin/add.c b/builtin/add.c\nindex c27254a5cd..5126d2ede3 100644\n--- a/builtin/add.c\n+++ b/builtin/add.c\n@@ -182,7 +182,7 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n \tgit_config(git_diff_basic_config, NULL); /* no \"diff\" UI options */\n \n \tif (repo_read_index(the_repository) < 0)\n-\t\tdie(_(\"Could not read the index\"));\n+\t\tdie(_(\"could not read the index\"));\n \n \trepo_init_revisions(the_repository, &rev, prefix);\n \trev.diffopt.context = 7;\n@@ -200,15 +200,15 @@ static int edit_patch(int argc, const char **argv, const char *prefix)\n \t\tdie(_(\"editing patch failed\"));\n \n \tif (stat(file, &st))\n-\t\tdie_errno(_(\"Could not stat '%s'\"), file);\n+\t\tdie_errno(_(\"could not stat '%s'\"), file);\n \tif (!st.st_size)\n-\t\tdie(_(\"Empty patch. Aborted.\"));\n+\t\tdie(_(\"empty patch. aborted\"));\n \n \tchild.git_cmd = 1;\n \tstrvec_pushl(&child.args, \"apply\", \"--recount\", \"--cached\", file,\n \t\t     NULL);\n \tif (run_command(&child))\n-\t\tdie(_(\"Could not apply '%s'\"), file);\n+\t\tdie(_(\"could not apply '%s'\"), file);\n \n \tunlink(file);\n \tfree(file);\n@@ -568,7 +568,7 @@ int cmd_add(int argc, const char **argv, const char *prefix)\n finish:\n \tif (write_locked_index(&the_index, &lock_file,\n \t\t\t       COMMIT_LOCK | SKIP_IF_UNCHANGED))\n-\t\tdie(_(\"Unable to write new index file\"));\n+\t\tdie(_(\"unable to write new index file\"));\n \n \tdir_clear(&dir);\n \tclear_pathspec(&pathspec);\n-- \n2.36.1.windows.1\n\n"},{"id":"483374","messageId":"xmqqttqox5cp.fsf@gitster.g","threadId":"60387","inReplyTo":"20231017221323.352-1-naomi.ibeh69@gmail.com","subject":"Re: [PATCH v2 1/1] [OUTREACHY] builtin/add.c: clean up die() messages","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2023-10-17T22:16:06Z","receivedAt":"2023-10-17T22:16:13Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Naomi Ibe <naomi.ibeh69@gmail.com> writes:\n\n> As described in the CodingGuidelines document, a single line\n> message given to die() and its friends should not capitalize its\n> first word, and should not add full-stop at the end.\n>\n> Signed-off-by: Naomi Ibe <naomi.ibeh69@gmail.com>\n> ---\n>  builtin/add.c | 10 +++++-----\n>  1 file changed, 5 insertions(+), 5 deletions(-)\n\nLooks good.  Will queue.  Thanks.\n"}]}