* [PATCH] prlimit: remove bogus instructions
@ 2012-04-14 10:01 Davidlohr Bueso
2012-04-23 10:07 ` Karel Zak
0 siblings, 1 reply; 2+ messages in thread
From: Davidlohr Bueso @ 2012-04-14 10:01 UTC (permalink / raw)
To: Karel Zak; +Cc: util-linux
From: Davidlohr Bueso <dave@gnu.org>
usage() is noreturn!
Signed-off-by: Davidlohr Bueso <dave@gnu.org>
---
sys-utils/prlimit.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/sys-utils/prlimit.c b/sys-utils/prlimit.c
index 12271a0..7e52a2d 100644
--- a/sys-utils/prlimit.c
+++ b/sys-utils/prlimit.c
@@ -585,7 +585,6 @@ int main(int argc, char **argv)
break;
case 'h':
usage(stdout);
- break;
case 'o':
ncolumns = string_to_idarray(optarg,
columns, ARRAY_SIZE(columns),
@@ -609,7 +608,6 @@ int main(int argc, char **argv)
default:
usage(stderr);
- break;
}
}
if (argc > optind && pid)
--
1.7.4.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] prlimit: remove bogus instructions
2012-04-14 10:01 [PATCH] prlimit: remove bogus instructions Davidlohr Bueso
@ 2012-04-23 10:07 ` Karel Zak
0 siblings, 0 replies; 2+ messages in thread
From: Karel Zak @ 2012-04-23 10:07 UTC (permalink / raw)
To: Davidlohr Bueso; +Cc: util-linux
On Sat, Apr 14, 2012 at 12:01:03PM +0200, Davidlohr Bueso wrote:
> sys-utils/prlimit.c | 2 --
> 1 files changed, 0 insertions(+), 2 deletions(-)
Applied, thanks.
BTW, some (including really expensive) static analyzers have no clue
about 'noreturn' function attribute :-)
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-04-23 10:07 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-04-14 10:01 [PATCH] prlimit: remove bogus instructions Davidlohr Bueso
2012-04-23 10:07 ` Karel Zak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox