Rewrite scoped_ptr<T>(NULL) to use the default ctor in chromeos/.

chromeos/dbus/debug_daemon_client.cc

Index: chromeos/dbus/debug_daemon_client.cc
diff --git a/chromeos/dbus/debug_daemon_client.cc b/chromeos/dbus/debug_daemon_client.cc
index 29c5b1c3d24102bd7476a5f7402226f502515d3b..19ebd279194977d90eacb0662e075940a93653f5 100644
--- a/chromeos/dbus/debug_daemon_client.cc
+++ b/chromeos/dbus/debug_daemon_client.cc
@@ -44,8 +44,7 @@ class PipeReader {
   typedef base::Callback<void(void)>IOCompleteCallback;
 
   explicit PipeReader(IOCompleteCallback callback)
-      : data_stream_(NULL),
-        io_buffer_(new net::IOBufferWithSize(4096)),
+      : io_buffer_(new net::IOBufferWithSize(4096)),
         callback_(callback),
         weak_ptr_factory_(this) {
     pipe_fd_[0] = pipe_fd_[1] = -1;
@@ -142,9 +141,7 @@ namespace chromeos {
 class DebugDaemonClientImpl : public DebugDaemonClient {
  public:
   explicit DebugDaemonClientImpl(dbus::Bus* bus)
-      : debugdaemon_proxy_(NULL),
-        pipe_reader_(NULL),
-        weak_ptr_factory_(this) {
+      : debugdaemon_proxy_(NULL), weak_ptr_factory_(this) {

[Daniel Erat, 2013/06/13 15:46:33]

nit: i believe that these should remain on separate lines since they don't fit
on the same line as the c'tor's signature

[dcheng, 2013/06/13 15:48:11]

This is the result from clang-format-diff. If you prefer, I can manually revert
this.

[Daniel Erat, 2013/06/13 15:49:44]

thanks, please do so. i think that clang-format-diff conflicts with the style
guide here -- see
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Constr...

[dcheng, 2013/06/13 18:49:20]

Done.

However, a literal reading suggests that this is, in fact, permissible.
"Constructor initializer lists can be all on one line"
DebugDaemonClientImpl(dbus::Bus* bus) is not part of the initializer list, thus
the initializer list is still on one line.

Indirectly, further support for this can be found in
http://google-styleguide.googlecode.com/svn/trunk/cppguide.xml?showone=Constr...;
there, it explicitly states that parameters should be on the same line if they
fit.

     debugdaemon_proxy_ = bus->GetObjectProxy(
         debugd::kDebugdServiceName,
         dbus::ObjectPath(debugd::kDebugdServicePath));