[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