Thread (5 messages) 5 messages, 2 authors, 2016-06-15

Re: [PATCH] Fix detection of uname failure

flat view

From: Charles Bailey <hidden>
Date: 2016-06-15 23:05:47

On Fri, Jul 17, 2015 at 03:06:57PM +0200, Johannes Schindelin wrote:
From a quick `git grep '== -1'` and another quick `git grep '< 0'` it appears to me that we prefer the latter. Maybe you want to adjust it in the patch, too?
I did the same grep and found lots of examples of both. Many of the "<
0" applied to comparisons with variables and not API calls and many were
internal (to git) calls and not POSIX or C library calls so I wasn't
convinced to change my initial fix.

Having said that and thought about it some more, I think '< 0' is
probably better. In POSIX, we shouldn't ever get a negative value which
isn't -1, but if we ever do it is probably safer to fail. I'll send and
update.

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