Use nullptr comparisons
authorKevin Sawicki <kevinsawicki@gmail.com>
Tue, 28 Jun 2016 22:28:27 +0000 (15:28 -0700)
committerKevin Sawicki <kevinsawicki@gmail.com>
Wed, 29 Jun 2016 18:52:58 +0000 (11:52 -0700)
atom/browser/ui/win/notify_icon.cc

index a360c92..a82ac36 100644 (file)
@@ -134,7 +134,7 @@ void NotifyIcon::DisplayBalloon(HICON icon,
 void NotifyIcon::PopUpContextMenu(const gfx::Point& pos,
                                   ui::SimpleMenuModel* menu_model) {
   // Returns if context menu isn't set.
-  if (!menu_model && !menu_model_)
+  if (menu_model == nullptr && menu_model_ == nullptr)
     return;
 
   // Set our window as the foreground window, so the context menu closes when
@@ -148,7 +148,7 @@ void NotifyIcon::PopUpContextMenu(const gfx::Point& pos,
     rect.set_origin(gfx::Screen::GetScreen()->GetCursorScreenPoint());
 
   views::MenuRunner menu_runner(
-      menu_model ? menu_model : menu_model_,
+      menu_model != nullptr ? menu_model : menu_model_,
       views::MenuRunner::CONTEXT_MENU | views::MenuRunner::HAS_MNEMONICS);
   ignore_result(menu_runner.RunMenuAt(
       NULL, NULL, rect, views::MENU_ANCHOR_TOPLEFT, ui::MENU_SOURCE_MOUSE));