system/serenity
system/serenity
1
0
Fork 0
mirror of https://github.com/SerenityOS/serenity synced 2024-10-07 00:19:27 +00:00
Code Issues Packages Projects Releases Wiki Activity

AK: Simplify JsonObject and JsonArray API a little bit

Browse source 
Instead of set(const JsonValue&) and set(JsonValue&&), just do
set(JsonValue) and let callers move() if they want. This removes some
ambiguity and the compiler is smart enough to optimize it anyway.
This commit is contained in:
Andreas Kling 2020-03-06 08:49:48 +01:00
parent 211e938234
commit dc039fdc7e
2 changed files with 2 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
AK/JsonArray.h
View file

@ -68,8 +68,7 @@ public:
const JsonValue& operator[](int index) const { return at(index); }
void clear() { m_values.clear(); }
void append(const JsonValue& value) { m_values.append(value); }
void append(JsonValue&& value) { m_values.append(move(value)); }
void append(JsonValue value) { m_values.append(move(value)); }
template<typename Builder>
typename Builder::OutputType serialized() const;

7
AK/JsonObject.h
View file

@ -85,16 +85,11 @@ public:
return m_members.contains(key);
}
void set(const String& key, JsonValue&& value)
void set(const String& key, JsonValue value)
{
m_members.set(key, move(value));
}
void set(const String& key, const JsonValue& value)
{
m_members.set(key, JsonValue(value));
}
template<typename Callback>
void for_each_member(Callback callback) const
{