diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-03-28 09:24:15 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-03-29 10:12:41 -0400 |
| commit | 8333c77edf8fd1654cd96f3f6ee0f078dd64b58b (patch) | |
| tree | 135da27f10763e512868c34155891864877f6622 /src/shared/calendarspec.c | |
| parent | 0db809489fd88a320ae1023ffe36a9965e9a91b2 (diff) | |
Always use errno > 0 to help gcc
gcc thinks that errno might be negative, and functions could return
something positive on error (-errno). Should not matter in practice,
but makes an -O4 build much quieter.
Diffstat (limited to 'src/shared/calendarspec.c')
| -rw-r--r-- | src/shared/calendarspec.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/shared/calendarspec.c b/src/shared/calendarspec.c index cc680779b5..13f70d8e72 100644 --- a/src/shared/calendarspec.c +++ b/src/shared/calendarspec.c @@ -391,7 +391,7 @@ static int prepend_component(const char **p, CalendarComponent **c) { errno = 0; value = strtoul(*p, &e, 10); - if (errno != 0) + if (errno > 0) return -errno; if (e == *p) return -EINVAL; @@ -400,7 +400,7 @@ static int prepend_component(const char **p, CalendarComponent **c) { if (*e == '/') { repeat = strtoul(e+1, &ee, 10); - if (errno != 0) + if (errno > 0) return -errno; if (ee == e+1) return -EINVAL; |