diff options
| author | Jordan Bayles <jophba@chromium.org> | 2019-12-06 14:47:48 -0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2019-12-06 23:22:58 +0000 |
| commit | 84b1341d27e60346914d43d7b8fb385eebba66bb (patch) | |
| tree | fe5fdb50d105859fdb2db568aa87ad4b96e2c92a /cast/streaming/receiver_session.h | |
| parent | ca24ee5f1e56c7b5d5a333630794f6ee8afaaf38 (diff) | |
| download | openscreen-84b1341d27e60346914d43d7b8fb385eebba66bb.tar.gz | |
Remove Json(Reader|Writer) in favor of namespace
In a recent code review, it was determined that we generally don't want
classes to store JsonWriter and JsonReader instances. Thus, there isn't
really a point in having instances of these classes.
This patch deletes both JsonReader and JsonWriter in favor of a new,
proper json:: namespace in json_serialization.h that includes both
json::Read and json::Write.
Change-Id: Ib31c84339f1bf18919764fd0297a60d34cb66573
Reviewed-on: https://chromium-review.googlesource.com/c/openscreen/+/1956136
Commit-Queue: Jordan Bayles <jophba@chromium.org>
Reviewed-by: Yuri Wiitala <miu@chromium.org>
Diffstat (limited to 'cast/streaming/receiver_session.h')
| -rw-r--r-- | cast/streaming/receiver_session.h | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/cast/streaming/receiver_session.h b/cast/streaming/receiver_session.h index dfb2ca6a..7237c911 100644 --- a/cast/streaming/receiver_session.h +++ b/cast/streaming/receiver_session.h @@ -14,7 +14,7 @@ #include "cast/streaming/receiver.h" #include "cast/streaming/receiver_packet_router.h" #include "cast/streaming/session_config.h" -#include "util/json/json_reader.h" +#include "util/json/json_serialization.h" namespace cast { @@ -126,7 +126,6 @@ class ReceiverSession final : public MessagePort::Client { const Preferences preferences_; ReceiverPacketRouter packet_router_; - openscreen::JsonReader json_reader_ = {}; }; } // namespace streaming |