← Back to team overview

linuxdcpp-team team mailing list archive

[Bug 521681] Re: Pass reason in disconnect signal

 

...done differently but it's there...

** Changed in: adchpp
       Status: New => Fix Committed

-- 
You received this bug notification because you are a member of
Dcplusplus-team, which is subscribed to ADCH++.
https://bugs.launchpad.net/bugs/521681

Title:
  Pass reason in disconnect signal

Status in ADCH++:
  Fix Committed

Bug description:
  It would be nice to change how disconnect reasons are handled. I would
  like the internal reason collection in Util to be removed and instead
  include them in the disconnect signal. As far as i can tell nothing is
  done with the collected disconnect stats in the core, only plugins use
  them. In the code proposal i provide i'v added a REASON_DEFAULT (to be
  used if nothing else has been set and i'v also set the Util reasons to
  start at 200 (so STA error codes could be used as reasons without
  conflicts)

  Providing a diff to more clearly show what i want.