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

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 );
                 }
         }