Submission for C++ Readability

chromeos/dbus/fake_bluetooth_profile_manager_client.cc

 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chromeos/dbus/fake_bluetooth_profile_manager_client.h"
 
-
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "chromeos/dbus/fake_bluetooth_profile_service_provider.h"
 #include "dbus/bus.h"
 #include "dbus/message.h"
 #include "dbus/object_proxy.h"
 #include "third_party/cros_system_api/dbus/service_constants.h"
 
 namespace chromeos {
 
@@ skipping 22 matching lines @@
   VLOG(1) << "RegisterProfile: " << profile_path.value() << ": " << uuid;
 
   if (uuid == kUnregisterableUuid) {
     base::MessageLoop::current()->PostTask(
         FROM_HERE, base::Bind(error_callback,
                               bluetooth_profile_manager::kErrorInvalidArguments,
                               "Can't register this UUID"));
     return;
   }
 
-  // check options for channel & psm
+  // TODO(jamuraa): check options for channel & psm
 
   ServiceProviderMap::iterator iter = service_provider_map_.find(profile_path);
   if (iter == service_provider_map_.end()) {
     error_callback.Run(bluetooth_profile_manager::kErrorInvalidArguments,
                        "No profile created");
   } else {
     ProfileMap::iterator piter = profile_map_.find(uuid);
     if (piter != profile_map_.end()) {
       error_callback.Run(bluetooth_profile_manager::kErrorAlreadyExists,
                          "Profile already registered");
@@ skipping 38 matching lines @@
       service_provider_map_.find(service_provider->object_path_);
   if (iter != service_provider_map_.end() && iter->second == service_provider)
     service_provider_map_.erase(iter);
 }
 
 FakeBluetoothProfileServiceProvider*
 FakeBluetoothProfileManagerClient::GetProfileServiceProvider(
     const std::string& uuid) {
   ProfileMap::iterator iter = profile_map_.find(uuid);
   if (iter == profile_map_.end())
-    return NULL;
+    return nullptr;
   return service_provider_map_[iter->second];
 }
 
 }  // namespace chromeos
+

[omoikane, 2015/06/23 01:23:40]

This trailing blank line is redundant and should be deleted.

(8 more files like this)