Changes in src/SymTab/Validate.cc [a436947:74d1804]

File:

• src/SymTab/Validate.cc (modified) (1 diff)

src/SymTab/Validate.cc

@@ -483 +483 @@
 
         void ReturnChecker::visit( ReturnStmt * returnStmt ) {
+                // Previously this also checked for the existence of an expr paired with no return values on
+                // the  function return type. This is incorrect, since you can have an expression attached to
+                // a return statement in a void-returning function in C. The expression is treated as if it
+                // were cast to void.
                 if ( returnStmt->get_expr() == NULL && returnVals.size() != 0 ) {
                         throw SemanticError( "Non-void function returns no values: " , returnStmt );
-                } else if ( returnStmt->get_expr() != NULL && returnVals.size() == 0 ) {
-                        throw SemanticError( "void function returns values: " , returnStmt );
                 }
         }