commit a0011d89287b4bdcb8fcc7f07071be1e94c20f90
author Ben Chan <benchan@chromium.org> Tue May 13 00:19:29 2014 -0700
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Wed Jun 18 23:44:04 2014 +0000
tree ad296554d258a44aad511faa75c11505593e104f
parent 4ca36ad3f6664d3ba464c4022cd1501c310c21a4

debugd: Fix coding style and cpplint issues.

BUG=chromium:385849
TEST=`FEATURES=test emerge-$BOARD platform2`
TEST=`cpplint.py --root=src --filter=-build/include src/{*.h,*.cc,*/*.cc}`

Change-Id: Ia2475d6ee287dc727c3b2ffe139e9cc8bfe2e2e5
Reviewed-on: https://chromium-review.googlesource.com/204391
Reviewed-by: Jorge Lucangeli Obes <jorgelo@chromium.org>
Tested-by: Ben Chan <benchan@chromium.org>
Commit-Queue: Ben Chan <benchan@chromium.org>

debugd/src/storage_tool.h

diff --git a/debugd/src/storage_tool.h b/debugd/src/storage_tool.h
index 3cbc0b2..366f005 100644
--- a/debugd/src/storage_tool.h
+++ b/debugd/src/storage_tool.h
@@ -2,8 +2,8 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef STORAGE_TOOL_H
-#define STORAGE_TOOL_H
+#ifndef STORAGE_TOOL_H_
+#define STORAGE_TOOL_H_
 
 #include <string>
 
@@ -17,16 +17,15 @@
 class StorageTool : public SubprocessTool {
  public:
   StorageTool();
-  ~StorageTool();
+  virtual ~StorageTool();
 
-  std::string Smartctl(const std::string& option,
-                       DBus::Error& error); // NOLINT
-  std::string Start(const DBus::FileDescriptor& outfd,
-                    DBus::Error& error);
+  std::string Smartctl(const std::string& option, DBus::Error* error);
+  std::string Start(const DBus::FileDescriptor& outfd, DBus::Error* error);
+
  private:
   DISALLOW_COPY_AND_ASSIGN(StorageTool);
 };
 
-};  // namespace debugd
+}  // namespace debugd
 
-#endif  // !STORAGE_TOOL_H
+#endif  // STORAGE_TOOL_H_