support : and ? in flow context as per updated YAML spec

dlang-community · Oct 6, 2023 · 8b18bb0 · 8b18bb0
1 parent d009f15
commit 8b18bb0
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 15 deletions.
diff --git a/docs/articles/spec_differences.md b/docs/articles/spec_differences.md
@@ -17,10 +17,6 @@ Differences that can cause valid YAML documents not to load:
   - No support for byte order marks and multiple Unicode encodings in a
     stream.
 
-  - Plain scalars in flow context cannot contain `,`, `:` and `?`. This
-    might change with `:` in the future. See
-    <http://pyyaml.org/wiki/YAMLColonInFlowContext> for details.
-
   - The specification does not restrict characters for anchors and
     aliases. This may lead to problems, for instance, the document:
 

diff --git a/source/dyaml/scanner.d b/source/dyaml/scanner.d
@@ -1541,23 +1541,15 @@ struct Scanner
                     const cNext = reader_.peek(length + 1);
                     if(c.isWhiteSpace ||
                        (flowLevel_ == 0 && c == ':' && cNext.isWhiteSpace) ||
-                       (flowLevel_ > 0 && c.among!(',', ':', '?', '[', ']', '{', '}')))
+                       (flowLevel_ > 0 && c == ':' && (cNext.isWhiteSpace || cNext.among!(',', '[', ']', '{', '}'))) ||
+                       (flowLevel_ > 0 && c.among!(',', '[', ']', '{', '}')))
                     {
                         break;
                     }
                     ++length;
                     c = cNext;
                 }
 
-                // It's not clear what we should do with ':' in the flow context.
-                enforce(flowLevel_ == 0 || c != ':' ||
-                   reader_.peek(length + 1).isWhiteSpace ||
-                   reader_.peek(length + 1).among!(',', '[', ']', '{', '}'),
-                    new ScannerException("While scanning a plain scalar", startMark,
-                        "found unexpected ':' . Please check " ~
-                        "http://pyyaml.org/wiki/YAMLColonInFlowContext for details.",
-                        reader_.mark));
-
                 if(length == 0) { break; }
 
                 allowSimpleKey_ = false;

diff --git a/test/data/colon-in-flow-context.canonical b/test/data/colon-in-flow-context.canonical
@@ -0,0 +1,3 @@
+%YAML 1.1
+---
+"foo:bar":
diff --git a/test/data/colon-in-flow-context.data b/test/data/colon-in-flow-context.data
@@ -0,0 +1,3 @@
+%YAML 1.1
+---
+{ foo:bar }
diff --git a/test/data/colon-in-flow-context.loader-error b/test/data/colon-in-flow-context.loader-error
diff --git a/test/data/question-mark-in-flow-context.canonical b/test/data/question-mark-in-flow-context.canonical
@@ -0,0 +1,3 @@
+%YAML 1.1
+---
+"foo?bar":
diff --git a/test/data/question-mark-in-flow-context.data b/test/data/question-mark-in-flow-context.data
@@ -0,0 +1,3 @@
+%YAML 1.1
+---
+{ foo?bar }