Thread (2 messages) 2 messages, 2 authors, 2016-08-11

Re: [PATCH] Documentation: Update information about <format> in git-for-each-ref

flat view

From: Andreas Ericsson <hidden>
Date: 2016-08-11 20:32:20

Junio C Hamano wrote:
Andreas Ericsson [off-list ref] writes:
quoted
Jakub Narebski wrote:
quoted
BTW. I had to translate
+       if (strcmp(who, "tagger") && strcmp(who, "committer"))
to
+       if (strcmp(who, "tagger") == 0 || strcmp(who, "committer") == 0)
to understand it. But this is probably my lack of contact with such
C idioms.
But this does the exact opposite....
"Compare" (as in "strcmp") also translates to "are equal to" and isn't
only a verb. This is unfortunate for people who aren't natively
english and has had me confused on many a long night...
Being a non-English speaker, I always pronounce xxxcmp() used as
boolean 'is different' in my head.  The (correct version of the)
above example expression is read as 'if it is different from
"tagger" and it is different from "committer", then do this'.
Without any slight intended toward your english proficiency, I'd say your programming is better than your english. For me it's the other way around, so I think of the above as "if it doesn't compare to this or that, then do this", but the "are equal to" meaning of "compare" isn't intuitive to me as I spent the first several years of my english-speaking life using "compare" exclusively as a verb.

And yes. There was a compliment hidden in there. Soak it up. You've earned it. ;-)

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help