Add purple_notification_manager_remove_with_account.

Review Request #1612 — Created Aug. 21, 2022 and submitted

Information

pidgin/pidgin
default

Reviewers

This is used when an account is disabled or removed to purge its notifications
from the manager.

Ran the unit tests and forced them to fail with bad values to verify they were correct.

Summary ID
Add purple_notification_manager_remove_with_account.
This is used when an account is disabled or removed to purge it's notifications from the manager.
a69bfd0a49f07e704755d76226cabe2e2a11a45c
Description From Last Updated

its

QuLogicQuLogic

I suppose this starts at 1 so that pattern can get 0 as a sentinel, but since it's a gint, …

QuLogicQuLogic
QuLogic
QuLogic
  1. 
      
  2. libpurple/purplenotificationmanager.c (Diff revision 1)
     
     
    Show all issues

    its

  3. libpurple/tests/test_notification_manager.c (Diff revision 1)
     
     
     
     
    Show all issues

    I suppose this starts at 1 so that pattern can get 0 as a sentinel, but since it's a gint, it could also get -1 as a sentinel?

    1. yeah, for some reason I was thinking it was a guint which is why I went with zero..

  4. 
      
grim
QuLogic
  1. Ship It!
  2. 
      
grim
Review request changed
Status:
Completed