Bug #747

[PATCH] Cleanup unused CoreSessions

Added by seezer over 8 years ago. Updated over 2 years ago.

Quassel Core
Target version:
Start date:
Due date:
% Done:




The attached patch detects if a CoreSession isn't of any use, deletes it and prevents it from beeing started automatically at next core startup.

A session is assumed useless if it has no clients attached and any irc network has been disconnected deliberately.

Until now, quasselcore starts a new thread for every user that once had a usefull session.

0001-Detect-and-kill-unused-CoreSessions.patch View (4.62 KB) seezer, 07/22/2009 12:04 PM

Related issues

Related to Quassel IRC - Feature #909: Provide --del-user argument New 02/07/2010


#1 Updated by johu over 7 years ago

  • Version changed from 0.5-pre to 0.5.0

where are the remaining 10%? Is there something new or it is already done?

#2 Updated by seezer over 7 years ago

johu wrote:

where are the remaining 10%? Is there something new or it is already done?

One of those things rotting in the bugtracker. It would have been merged in the time between featurefreeze and final release of 0.5.
Iirc some other core work produced merge conflicts by that time and i didn't want to introduce last-minute-core-bugs because the patch didn't get broad testing and i didn't receive any feedback. So i voted for skipping that patch for 0.5.
Since then I didn't look into that anymore. The problem still exists afaik.

As i still use a core with a series of patches on top of upstream, it's sometimes a bit messy to keep single patches like the above in my private core until it will or will not be merged some unpredictable time in the future :)
I hope to get some more quassel-time in a few weeks. If anyone is interested in that patch i could look into it again.

#3 Updated by Anonymous almost 6 years ago

  • Target version set to Some future release

Patch needs a bit of updating (some stuff moved in the headers). Seems to be useful still though.

#4 Updated by seezer over 2 years ago

  • Assignee deleted (seezer)


Also available in: Atom PDF