Changeset c11e31c for translator/ControlStruct/ForExprMutator.cc

Timestamp:

May 14, 2015, 1:44:55 PM (9 years ago)

Author:

Peter A. Buhr <pabuhr@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, ctor, deferred_resn, demangler, enum, forall-pointer-decay, gc_noraii, jacob/cs343-translation, jenkins-sandbox, master, memory, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, string, with_gc

Children:

4bf5298

Parents:

d4778a6

Message:

add inline and attribute qualifiers, cfa.y comment formatting, fix error message in isIntegralType

File:

• translator/ControlStruct/ForExprMutator.cc (modified) (2 diffs)

translator/ControlStruct/ForExprMutator.cc

@@ -5 +5 @@
 namespace ControlStruct {
     Statement *ForExprMutator::mutate( ForStmt *forStmt ) {
+        // recurse down all nest for loops to hoist any initializer declarations to make them C89 (rather than C99)
         forStmt->set_body( forStmt->get_body()->acceptMutator( *this ) );
         if ( DeclStmt *decl = dynamic_cast< DeclStmt * > ( forStmt->get_initialization() ) ) {
-            // create compound statement, move declaration outside, leave _for_ as-is
+            // create compound statement, move initializer declaration outside, leave _for_ as-is
             CompoundStmt *block = new CompoundStmt( std::list< Label >() );
             std::list<Statement *> &stmts = block->get_kids();
@@ -17 +18 @@
             return block;
         } // if
-        // ForStmt still needs to be fixed
-        else
-            return forStmt;
+
+        return forStmt;
     }
 } // namespace ControlStruct