Reworking of the code for notifications, highlights and ignores
After a small conversation with Tucos on the irc, it was pointed out that notifications, Highlights and Ignore are all implemented separately despite using the same underlying stuff.
I'd like to propose a rework the quassel notification system to unify Notifications/Highlight/Ignore on the back end to reduce code duplication/overhead and make it easier to implement new features in a consistent manner.
#1 Updated by genius3000 over 5 years ago
- Status changed from New to Resolved
- Target version set to 0.13.0
- % Done changed from 0 to 100
As of 0.13 (as of this comment, -rc1 is out), highlights are being transitioned to core-side (called remote highlights) and include highlight ignore rules as well. Another improvement is the unifying (maybe not 100% but quite a bit) of the text matching code for highlights and ignores along with regex caching to improve the speed of message checking and buffer switching significantly.
Reference to the unification/caching PR: https://github.com/quassel/quassel/pull/415
#2 Updated by digitalcircuit over 5 years ago
There's also some details on the Quassel wiki as to how the new pattern matching works: https://bugs.quassel-irc.org/projects/quassel-irc/wiki/Pattern_matching