commit 3d6b55295d49240a8e3575ffd8561231074714a9
author Vitaly Buka <vitalybuka@google.com> Tue Dec 22 13:14:16 2015 -0800
committer Vitaly Buka <vitalybuka@google.com> Wed Dec 23 01:53:36 2015 +0000
tree 36ede6964a9931e008ce3d8d28c7a33d48f32c1b
parent 66a01e0cd687c1e078b9166b61bf9112c70c1615

Hide Config::Load method

Constructor is going to load data instead.
This makes clear that ::Load is one time operations.

Change-Id: Id7b2275544cc034d30737bea21b3d8cc68e4a8a9
Reviewed-on: https://weave-review.googlesource.com/2080
Reviewed-by: Vitaly Buka <vitalybuka@google.com>

src/config.cc

diff --git a/src/config.cc b/src/config.cc
index 442a87a..37e907c 100644
--- a/src/config.cc
+++ b/src/config.cc
@@ -85,7 +85,9 @@
     : EnumToStringMap(kRootClientTokenOwnerMap) {}
 
 Config::Config(provider::ConfigStore* config_store)
-    : settings_{CreateDefaultSettings()}, config_store_{config_store} {}
+    : settings_{CreateDefaultSettings()}, config_store_{config_store} {
+  Load();
+}
 
 void Config::AddOnChangedCallback(const OnChangedCallback& callback) {
   on_changed_.push_back(callback);