shill: cleaning glint warnings
The typical warnings addressed include a lot of missing includes
and missing argument names. About 30 files edited in total.
Cpplint was the actual lint being run.
BUG=chromium-os:15873
TEST=Ran all the unit tests.
Change-Id: I64a8b76cd8f94c7729743b76b41a956f1b7370cf
Reviewed-on: https://gerrit.chromium.org/gerrit/12253
Commit-Ready: Hristo Stefanov <hstefanov@chromium.org>
Reviewed-by: Hristo Stefanov <hstefanov@chromium.org>
Tested-by: Hristo Stefanov <hstefanov@chromium.org>
diff --git a/property_accessor.h b/property_accessor.h
index 306f9c8..03fa7c0 100644
--- a/property_accessor.h
+++ b/property_accessor.h
@@ -101,8 +101,8 @@
// attempts to set via the accessor.
// It is an error to pass NULL for either of the other two arguments.
CustomAccessor(C *target,
- T(C::*getter)(Error *),
- void(C::*setter)(const T&, Error *))
+ T(C::*getter)(Error *error),
+ void(C::*setter)(const T &value, Error *error))
: target_(target),
getter_(getter),
setter_(setter) {
@@ -131,8 +131,8 @@
// Get() returns a const&, so we need to have internal storage to which to
// return a reference.
T storage_;
- T(C::*getter_)(Error *);
- void(C::*setter_)(const T&, Error *);
+ T(C::*getter_)(Error *error);
+ void(C::*setter_)(const T &value, Error *error);
DISALLOW_COPY_AND_ASSIGN(CustomAccessor);
};