From sgrubb at redhat.com  Thu Jul 20 23:02:10 2017
From: sgrubb at redhat.com (Steve Grubb)
Date: Thu, 20 Jul 2017 17:02:10 -0400
Subject: [Rd] [PATCH] Fix missing break
Message-ID: <60035856.27PnXlZqtm@x2>

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



[Rd] [PATCH] Fix missing break
Steve Grubb