Minor tidying to fit house style

`exit(1)` -> `exit(EXIT_FAILURE)`

Branch and loop bodies are always enclosed in `{` `}` even when they contain a single line

Author: hsutter

include/cpp2util.h

@@ -561,8 +561,7 @@ class contract_group {
         std::cerr << ": " << msg;
     }
     std::cerr << "\n";
-    // Get outta here but don't raise a signal
-    std::exit(1);
+    std::exit(EXIT_FAILURE);
 }
 
 auto inline cpp2_default = contract_group(

source/io.h

@@ -363,8 +363,9 @@ class braces_tracker
         }
 
         auto found_preprocessor_else_was_there_another() -> bool {
-            if (found_else)
+            if (found_else) {
                 return true; 
+            }
             found_else = true;
             return false;
         }

source/lex.h

@@ -718,13 +718,14 @@ auto lex_line(
         // the number of tokens has not gone down. So just push back as many
         // tokens as we merged. This will ensure that the token count remains
         // the same. 
-        for (auto i = 0; i < num_merged_tokens; i++)
+        for (auto i = 0; i < num_merged_tokens; i++) {
             tokens.push_back({
                 &generated_text.back()[0],
                 std::ssize(generated_text.back()),
                 pos,
                 lexeme::Keyword
                 });
+        }
 
         if (num_merged_tokens > 1)
         {

source/parse.h

@@ -9853,7 +9853,7 @@ class parser
                 curr().position(),
                 "invalid alias declaration - expected 'type', 'namespace', or a type-id after ':'"
             );
-        return {};
+            return {};
         }
 
         //  And the final ceremonial semicolon

source/sema.h

@@ -2052,8 +2052,9 @@ class sema
                 }
                 auto stmt_decl = stmt->get_if<declaration_node>();
                 //  If this is a declaration, check if it's a function
-                if (stmt_decl && stmt_decl->is_function())
+                if (stmt_decl && stmt_decl->is_function()) {
                     seen_function = true;
+                }
 
                 //  If this is called 'this', then make sure we haven't seen any functions
                 if (

source/to_cpp1.h

@@ -468,8 +468,9 @@ class positional_printer
                     )
                 {
                     print_comment( comments[next_comment] );
-                    if (!print_remaining_comments)
+                    if (!print_remaining_comments) {
                         assert(curr_pos.lineno <= pos.lineno);  // we shouldn't have overshot
+                    }
                 }
 
                 ++next_comment;
@@ -481,8 +482,9 @@ class positional_printer
             }
         }
         // And catch up.
-        while (curr_pos.lineno < pos.lineno)
+        while (curr_pos.lineno < pos.lineno) {
             print("\n");
+        }
     }
 
     auto print_unprinted_comments()