Hello,

There appears to be a break missing in the switch/case for the LISTSXP case.
If this is supposed to fall through, I'd suggest a comment so that others
know its by design.

Signed-off-by: Steve Grubb <sgr...@redhat.com>

Index: src/main/builtin.c
===================================================================
--- src/main/builtin.c  (revision 72935)
+++ src/main/builtin.c  (working copy)
@@ -888,6 +888,7 @@
            SETCAR(t, CAR(x));
            SET_TAG(t, TAG(x));
        }
+       break;
     case VECSXP:
        for (i = 0; i < len; i++)
            if (i < lenx) {

______________________________________________
R-devel@r-project.org mailing list
https://stat.ethz.ch/mailman/listinfo/r-devel

Reply via email to