diff options
| author | atatat <atatat@pkgsrc.org> | 2003-06-16 13:35:22 +0000 |
|---|---|---|
| committer | atatat <atatat@pkgsrc.org> | 2003-06-16 13:35:22 +0000 |
| commit | 49758757d0acd07ded60982d897b9822fcc21200 (patch) | |
| tree | 2b71140f93139e1c30e436fcb0f0567980c7717b | |
| parent | 9d330e06c2e999968c75ca297c5c56e315b585c8 (diff) | |
| download | pkgsrc-49758757d0acd07ded60982d897b9822fcc21200.tar.gz | |
A function that is declared to return an int *really ought to do do*.
I expect that since all possible code paths through this function did,
as their last step, an assignment of what was to be the return value,
that the registers somehow ended up nicely arranged after we fell out
of the bottom of the function.
But it's still better just to return a value.
| -rw-r--r-- | pkgtools/digest/files/digest.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/pkgtools/digest/files/digest.c b/pkgtools/digest/files/digest.c index f9798f4593a..92bb55cd35b 100644 --- a/pkgtools/digest/files/digest.c +++ b/pkgtools/digest/files/digest.c @@ -1,4 +1,4 @@ -/* $NetBSD: digest.c,v 1.6 2003/06/15 00:42:29 atatat Exp $ */ +/* $NetBSD: digest.c,v 1.7 2003/06/16 13:35:22 atatat Exp $ */ /* * Copyright (c) 2001 Alistair G. Crooks. All rights reserved. @@ -39,7 +39,7 @@ #ifndef lint __COPYRIGHT("@(#) Copyright (c) 2001 \ The NetBSD Foundation, Inc. All rights reserved."); -__RCSID("$NetBSD: digest.c,v 1.6 2003/06/15 00:42:29 atatat Exp $"); +__RCSID("$NetBSD: digest.c,v 1.7 2003/06/16 13:35:22 atatat Exp $"); #endif @@ -143,6 +143,8 @@ digest_file(char *fn, alg_t *alg) rc = 1; } } + + return (rc); } int |