Corrected off-by-one error.

nate-chandler · nate-chandler · commit f8ef2134752a · 2019-04-24T10:09:19.000-07:00
diff --git a/lib/Parse/ParseDecl.cpp b/lib/Parse/ParseDecl.cpp
@@ -5685,7 +5685,7 @@ void Parser::parseAbstractFunctionBody(AbstractFunctionDecl *AFD) {
         }
       } else if (auto *E = Element.dyn_cast<Expr *>()) {
         if (auto SE = dyn_cast<SequenceExpr>(E->getSemanticsProvidingExpr())) {
-          if (SE->getNumElements() > 0 && isa<AssignExpr>(SE->getElement(1))) {
+          if (SE->getNumElements() > 1 && isa<AssignExpr>(SE->getElement(1))) {
             // This is an assignment.  We don't want to implicitly return 
             // it.
             return;

Original file line number	Diff line number	Diff line change
`@@ -5685,7 +5685,7 @@ void Parser::parseAbstractFunctionBody(AbstractFunctionDecl *AFD) {`
`5685`	`5685`	`}`
`5686`	`5686`	`} else if (auto E = Element.dyn_cast<Expr >()) {`
`5687`	`5687`	`if (auto SE = dyn_cast<SequenceExpr>(E->getSemanticsProvidingExpr())) {`
`5688`		`- if (SE->getNumElements() > 0 && isa<AssignExpr>(SE->getElement(1))) {`
	`5688`	`+ if (SE->getNumElements() > 1 && isa<AssignExpr>(SE->getElement(1))) {`
`5689`	`5689`	`// This is an assignment. We don't want to implicitly return`
`5690`	`5690`	`// it.`
`5691`	`5691`	`return;`