Merge pull request #15851 from alalek:fixup_15842

pull/15866/head
Alexander Alekhin 5 years ago
commit 417034518c
  1. 17
      modules/core/src/persistence.cpp
  2. 3
      modules/core/src/persistence_yml.cpp
  3. 14
      modules/core/test/test_io.cpp

@ -1900,18 +1900,19 @@ int FileStorage::getFormat() const
FileNode FileStorage::operator [](const char* key) const
{
if( p->roots.empty() )
return FileNode();
return p->roots[0][key];
return this->operator[](std::string(key));
}
FileNode FileStorage::operator [](const std::string& key) const
{
if( p->roots.empty() )
return FileNode();
return p->roots[0][key];
FileNode res;
for (size_t i = 0; i < p->roots.size(); i++)
{
res = p->roots[i][key];
if (!res.empty())
break;
}
return res;
}
String FileStorage::releaseAndGetString()

@ -770,7 +770,7 @@ public:
bool first = true;
bool ok = true;
FileNode root_collection(fs->getFS(), 0, 0);
FileNode root_node = fs->addNode(root_collection, std::string(), FileNode::NONE);
for(;;)
{
// 0. skip leading comments and directives and ...
@ -821,6 +821,7 @@ public:
if( memcmp( ptr, "...", 3 ) != 0 )
{
// 2. parse the collection
FileNode root_node = fs->addNode(root_collection, std::string(), FileNode::NONE);
ptr = parseValue( ptr, root_node, 0, false );
if( !root_node.isMap() && !root_node.isSeq() )

@ -1654,9 +1654,19 @@ TEST(Core_InputOutput, FileStorage_YAML_parse_multiple_documents)
fs.release();
fs.open(filename, FileStorage::READ);
ASSERT_EQ(42, (int)fs["a"]);
ASSERT_EQ(1988, (int)fs["b"]);
EXPECT_EQ(42, (int)fs["a"]);
EXPECT_EQ(1988, (int)fs["b"]);
EXPECT_EQ(42, (int)fs.root(0)["a"]);
EXPECT_TRUE(fs.root(0)["b"].empty());
EXPECT_TRUE(fs.root(1)["a"].empty());
EXPECT_EQ(1988, (int)fs.root(1)["b"]);
fs.release();
ASSERT_EQ(0, std::remove(filename.c_str()));
}
}} // namespace

Loading…
Cancel
Save