[flang] Reformatting and some comment clean-up.

Original-commit: flang-compiler/f18@cedd3d82cf
Reviewed-on: https://github.com/flang-compiler/f18/pull/11

flang/lib/parser/grammar.h

@@ -8,9 +8,9 @@
 // library used here to implement an LL recursive descent recognizer.
 
 #include "basic-parsers.h"
-#include "token-parsers.h"
 #include "format-specification.h"
 #include "parse-tree.h"
+#include "token-parsers.h"
 #include "user-state.h"
 #include <cctype>
 #include <cinttypes>

flang/lib/parser/idioms.h

@@ -22,7 +22,7 @@
 #include <type_traits>
 #include <variant>
 
-// Avoid a deduction bug in GNU 7.1.0 headers by forcing the answer.
+// Avoid a deduction bug in GNU 7.3.0 headers by forcing the answer.
 // TODO: better resolution
 namespace std {
 template<typename A>

flang/lib/parser/parse-state.h

@@ -36,8 +36,7 @@ public:
       strictConformance_{that.strictConformance_},
       warnOnNonstandardUsage_{that.warnOnNonstandardUsage_},
       warnOnDeprecatedUsage_{that.warnOnDeprecatedUsage_},
-      anyErrorRecovery_{that.anyErrorRecovery_} {
-  }
+      anyErrorRecovery_{that.anyErrorRecovery_} {}
   ParseState(ParseState &&that)
     : cooked_{that.cooked_}, p_{that.p_}, limit_{that.limit_},
       column_{that.column_}, messages_{std::move(that.messages_)},
@@ -48,8 +47,7 @@ public:
       strictConformance_{that.strictConformance_},
       warnOnNonstandardUsage_{that.warnOnNonstandardUsage_},
       warnOnDeprecatedUsage_{that.warnOnDeprecatedUsage_},
-      anyErrorRecovery_{that.anyErrorRecovery_} {
-  }
+      anyErrorRecovery_{that.anyErrorRecovery_} {}
   ParseState &operator=(ParseState &&that) {
     swap(that);
     return *this;

flang/lib/parser/parse-tree.h

@@ -26,7 +26,7 @@
 
 // Parse tree node class types do not have default constructors.  They
 // explicitly declare "T() {} = delete;" to make this clear.  This restriction
-// avoids what would otherwise become a viral requirement to include
+// prevents the introduction of what would be a viral requirement to include
 // std::monostate among most std::variant<> discriminated union members.
 
 // Parse tree node class types do not have copy constructors or copy assignment

flang/lib/parser/token-parsers.h

@@ -414,8 +414,7 @@ struct ConsumedAllInputParser {
   }
 } consumedAllInput;
 
-template<char goal>
-struct SkipPast {
+template<char goal> struct SkipPast {
   using resultType = Success;
   constexpr SkipPast() {}
   constexpr SkipPast(const SkipPast &) {}