From: Linus Torvalds Date: Wed, 25 May 2005 01:24:58 GMT Subject: Re: [PATCH] diff-cache path restriction fix. Message-ID: In-Reply-To: <7vekbwru6x.fsf@assigned-by-dhcp.cox.net> On Tue, 24 May 2005, Junio C Hamano wrote: > >>>>> "LT" == Linus Torvalds writes: > > LT> No, it's more broken than that. > > I'll take a look at this later and submit an update, but an OT > point I feel I should address. I checked in the fixed arg parsing already ;) > The comparison lists things in the ascending order from left to > right. The fact that 1 comes before argc on that line of code > visually makes it obvious that I am talking about argc being > larger than one and that is the reason. I'd write (argc < 4) > not (4 > argc) for the same reason. Hmm. According to that logic, ">" and ">=" is superfluous. Also, what language do you actually speak? Every human language I know (admittedly, apart from Finnish they are all related) tends to say things like "if you have more than four children, you're in trouble", rather than saying "if four is less than the number of children you have". Notice? People compare something _to_ a number. They don't compare a number to something. We say "if more than four", we don't say "if four is less". Sadly, I can't get google to match "1 < x" and "x > 1" (it just considers all special characters to be basically whitespace, so "1 < x" matches "1 x" and "1.x" according to google. Usually google is a good way to get a feel for how common some phrase is, but not on things like this. Linus