commit ffa3d0433d419aa9251f3768a0090a27b6b1c434
author mukesh agrawal <quiche@chromium.org> Thu Oct 06 15:26:10 2011 -0700
committer mukesh agrawal <quiche@chromium.org> Fri Oct 07 09:52:15 2011 -0700
tree f7beac0b6eba964d1d0fa6fd25d7440342a8925b
parent ab565bb6f148849fc0ceb12529ff93acfbbed71f

shill: allow property accessors to return an error message if a
property's value cannot be changed.

also, reduce some code duplication in property_store.cc

BUG=chromium-os:21384
TEST=unittests

Change-Id: Iaac8d40bbb9e9a1341d6c6d01642885d88ac0e27
Reviewed-on: http://gerrit.chromium.org/gerrit/8925
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>

manager.h

diff --git a/manager.h b/manager.h
index ede4f49..289ee09 100644
--- a/manager.h
+++ b/manager.h
@@ -102,12 +102,14 @@
   std::vector<std::string> EnumerateWatchedServices();
   std::string GetActiveProfileName();
 
-  void HelpRegisterDerivedString(const std::string &name,
-                                 std::string(Manager::*get)(void),
-                                 bool(Manager::*set)(const std::string&));
-  void HelpRegisterDerivedStrings(const std::string &name,
-                                  Strings(Manager::*get)(void),
-                                  bool(Manager::*set)(const Strings&));
+  void HelpRegisterDerivedString(
+      const std::string &name,
+      std::string(Manager::*get)(void),
+      void(Manager::*set)(const std::string&, Error *));
+  void HelpRegisterDerivedStrings(
+      const std::string &name,
+      Strings(Manager::*get)(void),
+      void(Manager::*set)(const Strings&, Error *));
 
   bool OrderServices(ServiceRefPtr a, ServiceRefPtr b);
   void SortServices();