Code review comment for lp:~pete-woods/unity-notifications/handle-client-death

Revision history for this message
Charles Kerr (charlesk) wrote :

Looks good overall. Inline are one or two optional suggestions, plus one NF for how NotificationModel::removeAllNotificationsForClient() handles its loops.

review: Needs Fixing

« Back to merge proposal