zhangyiss/toml11
1
0
Fork 0
mirror of https://github.com/ToruNiina/toml11.git synced 2025-09-17 00:38:08 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

feat: keep only 1 scanner in a cache

Browse Source 
while parsing, thread_local cache will not use scanners constructed from
multiple toml::spec. we don't need to keep multiple scanners
This commit is contained in:
ToruNiina
2025-02-13 20:57:07 +09:00
parent 17c5599ad6
commit c4637a4222
1 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
include/toml11/impl/syntax_impl.hpp
View File

@@ -21,27 +21,21 @@ struct syntax_cache
static_assert(std::is_base_of<scanner_base, value_type>::value, "");
explicit syntax_cache(F f)
: func_(std::move(f)), cache_{}
: func_(std::move(f)), cache_(cxx::make_nullopt())
{}
value_type const& at(const spec& s)
{
const auto found = std::find_if(cache_.begin(), cache_.end(),
[&s](const std::pair<spec, value_type>& kv) { return kv.first == s; });
if(found == cache_.end())
if( ! this->cache_.has_value() || this->cache_.value().first != s)
{
this->cache_.emplace_back(s, func_(s));
return cache_.back().second;
}
else
{
return found->second;
this->cache_ = std::make_pair(s, func_(s));
}
return this->cache_.value().second;
}
private:
F func_;
std::vector<std::pair<spec, value_type>> cache_;
cxx::optional<std::pair<spec, value_type>> cache_;
};
template<typename F>