[llvm] r288015 - Revert "Improve error handling in YAML parsing"

[Mehdi Amini via llvm-commits]

Author: mehdi_amini
Date: Sun Nov 27 22:57:04 2016
New Revision: 288015

URL: http://llvm.org/viewvc/llvm-project?rev=288015&view=rev
Log:
Revert "Improve error handling in YAML parsing"

This reverts commit r288014, the unittest isn't passing

Modified:
    llvm/trunk/include/llvm/Support/YAMLTraits.h
    llvm/trunk/lib/Support/YAMLTraits.cpp
    llvm/trunk/unittests/Support/YAMLIOTest.cpp

Modified: llvm/trunk/include/llvm/Support/YAMLTraits.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Support/YAMLTraits.h?rev=288015&r1=288014&r2=288015&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Support/YAMLTraits.h (original)
+++ llvm/trunk/include/llvm/Support/YAMLTraits.h Sun Nov 27 22:57:04 2016
@@ -1085,7 +1085,6 @@ private:
   void scalarString(StringRef &, bool) override;
   void blockScalarString(StringRef &) override;
   void setError(const Twine &message) override;
-  bool hasError() const;
   bool canElideEmptySequence() override;
 
   class HNode {

Modified: llvm/trunk/lib/Support/YAMLTraits.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/YAMLTraits.cpp?rev=288015&r1=288014&r2=288015&view=diff
==============================================================================
--- llvm/trunk/lib/Support/YAMLTraits.cpp (original)
+++ llvm/trunk/lib/Support/YAMLTraits.cpp Sun Nov 27 22:57:04 2016
@@ -112,7 +112,7 @@ bool Input::mapTag(StringRef Tag, bool D
 }
 
 void Input::beginMapping() {
-  if (hasError())
+  if (EC)
     return;
   // CurrentNode can be null if the document is empty.
   MapHNode *MN = dyn_cast_or_null<MapHNode>(CurrentNode);
@@ -124,7 +124,7 @@ void Input::beginMapping() {
 bool Input::preflightKey(const char *Key, bool Required, bool, bool &UseDefault,
                          void *&SaveInfo) {
   UseDefault = false;
-  if (hasError())
+  if (EC)
     return false;
 
   // CurrentNode is null for empty documents, which is an error in case required
@@ -159,7 +159,7 @@ void Input::postflightKey(void *saveInfo
 }
 
 void Input::endMapping() {
-  if (hasError())
+  if (EC)
     return;
   // CurrentNode can be null if the document is empty.
   MapHNode *MN = dyn_cast_or_null<MapHNode>(CurrentNode);
@@ -196,7 +196,7 @@ void Input::endSequence() {
 }
 
 bool Input::preflightElement(unsigned Index, void *&SaveInfo) {
-  if (hasError())
+  if (EC)
     return false;
   if (SequenceHNode *SQ = dyn_cast<SequenceHNode>(CurrentNode)) {
     SaveInfo = CurrentNode;
@@ -213,7 +213,7 @@ void Input::postflightElement(void *Save
 unsigned Input::beginFlowSequence() { return beginSequence(); }
 
 bool Input::preflightFlowElement(unsigned index, void *&SaveInfo) {
-  if (hasError())
+  if (EC)
     return false;
   if (SequenceHNode *SQ = dyn_cast<SequenceHNode>(CurrentNode)) {
     SaveInfo = CurrentNode;
@@ -271,7 +271,7 @@ bool Input::beginBitSetScalar(bool &DoCl
 }
 
 bool Input::bitSetMatch(const char *Str, bool) {
-  if (hasError())
+  if (EC)
     return false;
   if (SequenceHNode *SQ = dyn_cast<SequenceHNode>(CurrentNode)) {
     unsigned Index = 0;
@@ -293,7 +293,7 @@ bool Input::bitSetMatch(const char *Str,
 }
 
 void Input::endBitSetScalar() {
-  if (hasError())
+  if (EC)
     return;
   if (SequenceHNode *SQ = dyn_cast<SequenceHNode>(CurrentNode)) {
     assert(BitValuesUsed.size() == SQ->Entries.size());
@@ -321,8 +321,6 @@ void Input::setError(HNode *hnode, const
   this->setError(hnode->_node, message);
 }
 
-bool Input::hasError() const { return EC || Strm->failed(); }
-
 void Input::setError(Node *node, const Twine &message) {
   Strm->printError(node, message);
   EC = make_error_code(errc::invalid_argument);
@@ -344,7 +342,7 @@ std::unique_ptr<Input::HNode> Input::cre
     auto SQHNode = llvm::make_unique<SequenceHNode>(N);
     for (Node &SN : *SQ) {
       auto Entry = this->createHNodes(&SN);
-      if (hasError())
+      if (EC)
         break;
       SQHNode->Entries.push_back(std::move(Entry));
     }
@@ -365,7 +363,7 @@ std::unique_ptr<Input::HNode> Input::cre
         KeyStr = StringStorage.str().copy(StringAllocator);
       }
       auto ValueHNode = this->createHNodes(KVN.getValue());
-      if (hasError())
+      if (EC)
         break;
       mapHNode->Mapping[KeyStr] = std::move(ValueHNode);
     }

Modified: llvm/trunk/unittests/Support/YAMLIOTest.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/unittests/Support/YAMLIOTest.cpp?rev=288015&r1=288014&r2=288015&view=diff
==============================================================================
--- llvm/trunk/unittests/Support/YAMLIOTest.cpp (original)
+++ llvm/trunk/unittests/Support/YAMLIOTest.cpp Sun Nov 27 22:57:04 2016
@@ -2368,11 +2368,3 @@ TEST(YAMLIO, TestMapWithContext) {
             out);
   out.clear();
 }
-
-TEST(YAMLIO, InvalidInput) {
-  // polluting 1 value in the sequence
-  Input yin("---\n- foo:  3\n  bar:  5\n1\n- foo:  3\n  bar:  5\n...\n");
-  std::vector<FooBar> Data;
-  yin >> Data;
-  EXPECT_TRUE((bool)yin.error());
-}