Another finding by IRIX cc: non-void function
Jarkko Hietaniemi [Wed, 12 Mar 2003 10:59:03 +0000 (10:59 +0000)]
sv_cat_decode() should return a value.

p4raw-id: //depot/perl@18954

sv.c

diff --git a/sv.c b/sv.c
index 955153a..4f6d59c 100644 (file)
--- a/sv.c
+++ b/sv.c
@@ -11499,8 +11499,8 @@ bool
 Perl_sv_cat_decode(pTHX_ SV *dsv, SV *encoding,
                   SV *ssv, int *offset, char *tstr, int tlen)
 {
+    bool ret = FALSE;
     if (SvPOK(ssv) && SvPOK(dsv) && SvROK(encoding) && offset) {
-        bool ret = FALSE;
        SV *offsv;
        dSP;
        ENTER;
@@ -11521,8 +11521,9 @@ Perl_sv_cat_decode(pTHX_ SV *dsv, SV *encoding,
        PUTBACK;
        FREETMPS;
        LEAVE;
-       return ret;
     }
-    Perl_croak(aTHX_ "Invalid argument to sv_cat_decode.");
+    else
+        Perl_croak(aTHX_ "Invalid argument to sv_cat_decode");
+    return ret;
 }