[llvm] [YAMLParser] Improve plain scalar spec compliance (PR #68946)

Scott Linder via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 16 15:52:29 PDT 2023


================
@@ -3153,7 +3153,7 @@ TEST(YAMLIO, TestFlowSequenceTokenErrors) {
 
 TEST(YAMLIO, TestDirectiveMappingNoValue) {
   Input yin("%YAML\n{5:");
-  EXPECT_FALSE(yin.setCurrentDocument());
+  yin.setCurrentDocument();
----------------
slinder1 wrote:

Is there a reason not to check the `setCurrentDocument` return value? Is it changing with the patch?

https://github.com/llvm/llvm-project/pull/68946


More information about the llvm-commits mailing list