[Rd] [PATCH] Fix missing break

Steve Grubb sgrubb at redhat.com
Thu Jul 20 23:02:10 CEST 2017


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 <sgrubb at 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) {



More information about the R-devel mailing list