Changeset 937e51d for src/ControlStruct/ChooseMutator.cc

Timestamp:

Jun 26, 2015, 4:00:26 PM (10 years ago)

Author:

Aaron Moss <a3moss@…>

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:

0df292b, e0ff3e6

Parents:

eb50842 (diff), 1869adf (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge pointer to pointer to qualified fix into master

File:

• src/ControlStruct/ChooseMutator.cc (modified) (2 diffs)

src/ControlStruct/ChooseMutator.cc

@@ -9 +9 @@
 // Author           : Rodolfo G. Esteves
 // Created On       : Mon May 18 07:44:20 2015
-// Last Modified By : Peter A. Buhr
-// Last Modified On : Tue May 19 15:31:39 2015
-// Update Count     : 2
+// Last Modified By : Rob Schluntz
+// Last Modified On : Wed Jun 03 15:30:20 2015
+// Update Count     : 5
 //
 
@@ -44 +44 @@
                 std::list< Statement * > &stmts = caseStmt->get_statements();
 
+                // the difference between switch and choose is that switch has an implicit fallthrough
+                // to the next case, whereas choose has an implicit break at the end of the current case.
+                // thus to transform a choose statement into a switch, we only need to insert breaks at the
+                // end of any case that doesn't already end in a break and that doesn't end in a fallthru
+
                 if ( insideChoose ) {
                         BranchStmt *posBrk;