Bring the YAML package's style up to modern standards.

pkg/yaml/lib/src/parser.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
-library parser;
+library yaml.parser;
 
 import 'dart:collection';
 
 import 'model.dart';
 import 'yaml_exception.dart';
 import 'yaml_map.dart';
 
 /// Translates a string of characters into a YAML serialization tree.
 ///
 /// This parser is designed to closely follow the spec. All productions in the
@@ skipping 384 matching lines @@
       end = _pos;
       return false;
     });
     _errorAnnotations[new _Range(start, end)] = message;
   }
 
   /// Throws an error with additional context information.
   error(String message) {
     // Line and column should be one-based.
     throw new SyntaxError(_line + 1, _column + 1,
-        "$message (in $_farthestContext)");
+        "$message (in $_farthestContext).");
   }
 
   /// If [result] is falsey, throws an error saying that [expected] was
   /// expected.
   expect(result, String expected) {
     if (truth(result)) return result;
-    error("expected $expected");
+    error("Expected $expected");
   }
 
   /// Throws an error saying that the parse failed. Uses [_farthestLine],
   /// [_farthestColumn], and [_farthestContext] to provide additional information.

[Bob Nystrom, 2014/05/09 00:18:56]

Long line.

[nweiz, 2014/05/20 00:15:07]

Done.

   parseFailed() {
-    var message = "invalid YAML in $_farthestContext";
+    var message = "Invalid YAML in $_farthestContext";
     var extraError = _errorAnnotations[_farthestPos];
     if (extraError != null) message = "$message ($extraError)";
-    throw new SyntaxError(_farthestLine + 1, _farthestColumn + 1, message);
+    throw new SyntaxError(_farthestLine + 1, _farthestColumn + 1, "$message.");
   }
 
   /// Returns the number of spaces after the current position.
   int countIndentation() {
     var i = 0;
     while (peek(i) == SP) i++;
     return i;
   }
 
   /// Returns the indentation for a block scalar.
@@ skipping 344 matching lines @@
   bool s_separate(int indent, int ctx) {
     switch (ctx) {
     case BLOCK_OUT:
     case BLOCK_IN:
     case FLOW_OUT:
     case FLOW_IN:
       return s_separateLines(indent);
     case BLOCK_KEY:
     case FLOW_KEY:
       return s_separateInLine();
-    default: throw 'invalid context "$ctx"';
+    default: throw 'Invalid context "$ctx".';
     }
   }
 
   // 81
   bool s_separateLines(int indent) {
     return transaction(() => s_l_comments() && s_flowLinePrefix(indent)) ||
       s_separateInLine();
   }
 
   // 82
@@ skipping 205 matching lines @@
       () => s_singleNextLine(indent),
       () => zeroOrMore(() => consume(isSpace))
     ]);
   }
 
   // 126
   bool ns_plainFirst(int ctx) {
     var char = peek();
     var indicator = indicatorType(char);
     if (indicator == C_RESERVED) {
-      error("reserved indicators can't start a plain scalar");
+      error("Reserved indicators can't start a plain scalar");

[Bob Nystrom, 2014/05/09 00:18:56]

"."

[nweiz, 2014/05/20 00:15:07]

[error] automatically adds a period.

     }
     var match = (isNonSpace(char) && indicator == null) ||
       ((indicator == C_MAPPING_KEY ||
         indicator == C_MAPPING_VALUE ||
         indicator == C_SEQUENCE_ENTRY) &&
        isPlainSafe(ctx, peek(1)));
 
     if (match) next();
     return match;
   }
 
   // 127
   bool isPlainSafe(int ctx, int char) {
     switch (ctx) {
     case FLOW_OUT:
     case BLOCK_KEY:
       // 128
       return isNonSpace(char);
     case FLOW_IN:
     case FLOW_KEY:
       // 129
       return isNonSpace(char) && !isFlowIndicator(char);
-    default: throw 'invalid context "$ctx"';
+    default: throw 'Invalid context "$ctx".';
     }
   }
 
   // 130
   bool ns_plainChar(int ctx) {
     var char = peek();
     var indicator = indicatorType(char);
     var safeChar = isPlainSafe(ctx, char) && indicator != C_MAPPING_VALUE &&
       indicator != C_COMMENT;
     var nonCommentHash = isNonSpace(peek(-1)) && indicator == C_COMMENT;
     var nonMappingColon = indicator == C_MAPPING_VALUE &&
       isPlainSafe(ctx, peek(1));
     var match = safeChar || nonCommentHash || nonMappingColon;
 
     if (match) next();
     return match;
   }
 
   // 131
   String ns_plain(int indent, int ctx) => context('plain scalar', () {
     return captureString(() {
       switch (ctx) {
       case FLOW_OUT:
       case FLOW_IN:
         return ns_plainMultiLine(indent, ctx);
       case BLOCK_KEY:
       case FLOW_KEY:
         return ns_plainOneLine(ctx);
-      default: throw 'invalid context "$ctx"';
+      default: throw 'Invalid context "$ctx".';
       }
     });
   });
 
   // 132
   void nb_ns_plainInLine(int ctx) {
     zeroOrMore(() => transaction(() {
       zeroOrMore(() => consume(isSpace));
       return ns_plainChar(ctx);
     }));
@@ skipping 19 matching lines @@
   // 135
   bool ns_plainMultiLine(int indent, int ctx) {
     if (!truth(ns_plainOneLine(ctx))) return false;
     zeroOrMore(() => s_ns_plainNextLine(indent, ctx));
     return true;
   }
 
   // 136
   int inFlow(int ctx) {
     switch (ctx) {
-    case FLOW_OUT:
-    case FLOW_IN:
-      return FLOW_IN;
-    case BLOCK_KEY:
-    case FLOW_KEY:
-      return FLOW_KEY;
+      case FLOW_OUT:
+      case FLOW_IN:
+        return FLOW_IN;
+      case BLOCK_KEY:
+      case FLOW_KEY:
+        return FLOW_KEY;
     }
+    throw "unreachable";
   }
 
   // 137
   SequenceNode c_flowSequence(int indent, int ctx) => transaction(() {
     if (!truth(c_indicator(C_SEQUENCE_START))) return null;
     zeroOrOne(() => s_separate(indent, ctx));
     var content = zeroOrOne(() => ns_s_flowSeqEntries(indent, inFlow(ctx)));
     if (!truth(c_indicator(C_SEQUENCE_END))) return null;
     return new SequenceNode("?", new List<Node>.from(content));
   });
@@ skipping 301 matching lines @@
   // 165
   bool b_chompedLast(int chomping) {
     if (atEndOfFile) return true;
     switch (chomping) {
     case CHOMPING_STRIP:
       return b_nonContent();
     case CHOMPING_CLIP:
     case CHOMPING_KEEP:
       return b_asLineFeed();
     }
+    throw "unreachable";
   }
 
   // 166
   void l_chompedEmpty(int indent, int chomping) {
     switch (chomping) {
     case CHOMPING_STRIP:
     case CHOMPING_CLIP:
       l_stripEmpty(indent);
       break;
     case CHOMPING_KEEP:
@@ skipping 489 matching lines @@
     for (var pair in _contents.reversed) {
       if (pair.first.contains(pos)) return pair.last;
     }
     return null;
   }
 
   /// Associates [value] with [range].
   operator[]=(_Range range, E value) =>
     _contents.add(new _Pair<_Range, E>(range, value));
 }