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

refactor: use is_something instead of is(...)

Browse Source 
to reduce the code size a bit
This commit is contained in:
ToruNiina
2019-05-29 21:20:22 +09:00
parent 1b78f161f5
commit 65124a8d2e
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
toml/serializer.hpp
View File

@@ -157,7 +157,7 @@ struct serializer
std::string operator()(const array& v) const std::string operator()(const array& v) const
{ {
if(!v.empty() && v.front().is(value_t::Table))// v is an array of tables if(!v.empty() && v.front().is_table())// v is an array of tables
{ {
// if it's not inlined, we need to add `[[table.key]]`. // if it's not inlined, we need to add `[[table.key]]`.
// but if it can be inlined, we need `table.key = [...]`. // but if it can be inlined, we need `table.key = [...]`.
@@ -411,7 +411,7 @@ struct serializer
// remaining non-table values will be assigned into [foo.bar], not [foo] // remaining non-table values will be assigned into [foo.bar], not [foo]
for(const auto kv : v) for(const auto kv : v)
{ {
if(kv.second.is(value_t::Table) || is_array_of_tables(kv.second)) if(kv.second.is_table() || is_array_of_tables(kv.second))
{ {
continue; continue;
} }
@@ -438,7 +438,7 @@ struct serializer
bool multiline_table_printed = false; bool multiline_table_printed = false;
for(const auto& kv : v) for(const auto& kv : v)
{ {
if(!kv.second.is(value_t::Table) && !is_array_of_tables(kv.second)) if(!kv.second.is_table() && !is_array_of_tables(kv.second))
{ {
continue; // other stuff are already serialized. skip them. continue; // other stuff are already serialized. skip them.
} }
@@ -467,10 +467,10 @@ struct serializer
bool is_array_of_tables(const value& v) const bool is_array_of_tables(const value& v) const
{ {
if(!v.is(value_t::Array)) {return false;} if(!v.is_array()) {return false;}
const auto& a = v.cast<value_t::Array>(); const auto& a = v.cast<value_t::Array>();
return !a.empty() && a.front().is(value_t::Table); return !a.empty() && a.front().is_table();
} }
private: private: