A few account manager changes

Review Request #2501 — Created July 7, 2023 and submitted

Information

pidgin/pidgin
default

Reviewers

  • Make the account manager add a new reference to the accounts. Previously this
    took the caller's reference, but some stuff was assuming it still owned it
    and we were leaking the reference if the caller didn't unref it themselves.
  • Disconnect our signal handlers for accounts when they are removed.
  • Add propagation signals for account::connected and account::disconnected.

Ran the unit tests and connected a few accounts without issues.

Summary ID
A few account manager changes
* Make the account manager add a new reference to the accounts. Previously this took the caller's reference, but some stuff was assuming it still owned it and we were leaking the reference if the caller didn't unref it themselves. * Add propagation signals for account::connected and account::disconnected.
8248952a192ef271f08eafad93cfce54ad1419ed
grim
ivanhoe
  1. Ship It!
  2. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...