mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Address cpplint issue "Constructors callable with one argument should be marked explicit. [runtime/explicit] [5]"

Browse source
This commit is contained in:
Ales Pergl 2017-04-05 13:20:16 +02:00
parent 1a7d9a5d20
commit 3336e02769
2 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
brightray/browser/win/win32_desktop_notifications/desktop_notification_controller.cc
View file

@ -260,7 +260,7 @@ DesktopNotificationController::Notification
data->image = CopyBitmap(image);
// Enqueue new notification
Notification ret = *queue_.insert(queue_.end(), move(data));
Notification ret { *queue_.insert(queue_.end(), move(data)) };
CheckQueue();
return ret;
}

6
brightray/browser/win/win32_desktop_notifications/desktop_notification_controller.h
View file

@ -11,7 +11,7 @@ struct NotificationData;
class DesktopNotificationController {
public:
DesktopNotificationController(unsigned maximumToasts = 3);
explicit DesktopNotificationController(unsigned maximumToasts = 3);
~DesktopNotificationController();
class Notification;
@ -43,7 +43,7 @@ class DesktopNotificationController {
// the `controller` member is cleared when the controller object
// stops tracking the notification
struct NotificationLink : std::shared_ptr<NotificationData> {
NotificationLink(DesktopNotificationController* controller);
explicit NotificationLink(DesktopNotificationController* controller);
~NotificationLink();
NotificationLink(NotificationLink&&) = default;
@ -90,7 +90,7 @@ class DesktopNotificationController {
class DesktopNotificationController::Notification {
public:
Notification() = default;
Notification(const std::shared_ptr<NotificationData>& data);
explicit Notification(const std::shared_ptr<NotificationData>& data);
bool operator==(const Notification& other) const;