Skip to content

Commit

Permalink
dbus: Fix clang warnings about missing virtual and OVERRIDE annotations.
Browse files Browse the repository at this point in the history
BUG=115047
TBR=satorux@chromium.org

Review URL: https://chromiumcodereview.appspot.com/10900041

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@154116 0039d316-1c4b-4281-b951-d872f2087c98
  • Loading branch information
tfarina@chromium.org committed Aug 30, 2012
1 parent ca3bea8 commit e24aa7d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions dbus/bus.cc
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ class Watch : public base::MessagePumpLibevent::Watcher {
dbus_watch_set_data(raw_watch_, this, NULL);
}

~Watch() {
virtual ~Watch() {
dbus_watch_set_data(raw_watch_, NULL, NULL);
}

Expand Down Expand Up @@ -74,13 +74,13 @@ class Watch : public base::MessagePumpLibevent::Watcher {

private:
// Implement MessagePumpLibevent::Watcher.
virtual void OnFileCanReadWithoutBlocking(int file_descriptor) {
virtual void OnFileCanReadWithoutBlocking(int file_descriptor) OVERRIDE {
const bool success = dbus_watch_handle(raw_watch_, DBUS_WATCH_READABLE);
CHECK(success) << "Unable to allocate memory";
}

// Implement MessagePumpLibevent::Watcher.
virtual void OnFileCanWriteWithoutBlocking(int file_descriptor) {
virtual void OnFileCanWriteWithoutBlocking(int file_descriptor) OVERRIDE {
const bool success = dbus_watch_handle(raw_watch_, DBUS_WATCH_WRITABLE);
CHECK(success) << "Unable to allocate memory";
}
Expand Down

0 comments on commit e24aa7d

Please sign in to comment.