diff options
author | Jim Meyering <meyering@redhat.com> | 2010-03-25 11:36:50 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2010-03-25 13:54:17 +0100 |
commit | a0e3040fd80528163565779a85a95b80469b7dfa (patch) | |
tree | af8b54c1d86cd9e8801cefaeec83b5f5798c0a76 | |
parent | 8f08d8e282bf3d4917e94135e67ba17d738313ce (diff) | |
download | grep-a0e3040fd80528163565779a85a95b80469b7dfa.tar.gz |
build: avoid warnings about unnecessary use of "return"
* src/grep.c (Gcompile, Ecompile, Acompile): Do not "return X"
from a function returning void, not even when X itself is a
function returning void. This avoids warnings from Sun Studio 11
reported by Dagobert Michelsen.
* src/egrep.c (Ecompile): Likewise.
-rw-r--r-- | src/egrep.c | 2 | ||||
-rw-r--r-- | src/grep.c | 12 |
2 files changed, 7 insertions, 7 deletions
diff --git a/src/egrep.c b/src/egrep.c index 80595fe5..f6a1d596 100644 --- a/src/egrep.c +++ b/src/egrep.c @@ -4,7 +4,7 @@ static void Ecompile (char const *pattern, size_t size) { - return GEAcompile (pattern, size, RE_SYNTAX_POSIX_EGREP | RE_NO_EMPTY_RANGES); + GEAcompile (pattern, size, RE_SYNTAX_POSIX_EGREP | RE_NO_EMPTY_RANGES); } struct matcher const matchers[] = { @@ -4,22 +4,22 @@ static void Gcompile (char const *pattern, size_t size) { - return GEAcompile (pattern, size, - RE_SYNTAX_GREP - | RE_HAT_LISTS_NOT_NEWLINE - | RE_NO_EMPTY_RANGES); + GEAcompile (pattern, size, + RE_SYNTAX_GREP + | RE_HAT_LISTS_NOT_NEWLINE + | RE_NO_EMPTY_RANGES); } static void Ecompile (char const *pattern, size_t size) { - return GEAcompile (pattern, size, RE_SYNTAX_POSIX_EGREP | RE_NO_EMPTY_RANGES); + GEAcompile (pattern, size, RE_SYNTAX_POSIX_EGREP | RE_NO_EMPTY_RANGES); } static void Acompile (char const *pattern, size_t size) { - return GEAcompile (pattern, size, RE_SYNTAX_AWK); + GEAcompile (pattern, size, RE_SYNTAX_AWK); } struct matcher const matchers[] = { |