Thread (12 messages) flat view 12 messages, 5 authors, 2016-06-15

Re: [PATCH] For Real - Fixed pluralization in diff reports

From: Sverre Rabbelier <hidden>
Date: 2016-06-15 22:51:42

Heya,

On Mon, Aug 1, 2011 at 06:46, Jon Forrest [off-list ref] wrote:
       fprintf(options->file, "%s", line_prefix);
       fprintf(options->file,
-              " %d files changed, %d insertions(+), %d deletions(-)\n",
-              total_files, adds, dels);
+              " %d file%s changed, %d insertion%s(+), %d deletion%s(-)\n",
+              total_files, total_files == 1 ? "" : "s", adds, adds == 1 ?
"" : "s", dels,
+               dels == 1 ? "" : "s");
 }
Also, this is rather detrimental to the i18n effort methinks?

-- 
Cheers,

Sverre Rabbelier
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help