Merge pull request #965 from rsjaffe/master

Added break to each case in switch statement
9 years ago · d666de70a3
parent 22c305fdee 24fa6d4b00
commit d666de70a3
1 changed files with 3 additions and 3 deletions
--- a/CppCoreGuidelines.md
+++ b/CppCoreGuidelines.md
@ -2206,9 +2206,9 @@ We might write
    bool owned;
    owner<istream*> inp;
    switch (source) {
-    case std_in:        owned = false; inp = &cin;
-    case command_line:  owned = true;  inp = new istringstream{argv[2]};
-    case file:          owned = true;  inp = new ifstream{argv[2]};
+    case std_in:        owned = false; inp = &cin;                       break;
+    case command_line:  owned = true;  inp = new istringstream{argv[2]}; break;
+    case file:          owned = true;  inp = new ifstream{argv[2]};      break;
    }
    istream& in = *inp;