[PATCH] D42362: [YAML] Plain scalars can not begin with most indicators.

[Sam McCall via Phabricator via cfe-commits]

sammccall accepted this revision.
sammccall added inline comments.
This revision is now accepted and ready to land.


================
Comment at: include/llvm/Support/YAMLTraits.h:524
+  // ambiguity with other YAML constructs.
+  // FIXME: However, the “:”, “?” and “-” indicators may be used as the first
+  // character if followed by a non-space “safe” character, as this causes no
----------------
Up to you, but I'd leave this comment out, or rephrase as a non-fixme - there's nothing *wrong* with the new behavior.


Repository:
  rL LLVM

https://reviews.llvm.org/D42362