Using the <QtCore> and <QtGui> includes is bad, because it introduces unnecessary slowness during
compilation; also, Qt5 moved some stuff around (e.g. from QtGui to QtWidgets).
So remove these, and add the proper #includes where needed instead.
Using the memory address of the QVariant that acts as buffer is bad. While we got
lucky with Qt4, and this worked by accident, it leads to subtle and not-so-subtle segfaults
when used with Qt5.
Instead, using QVariant::data() makes sure that implicitly shared data is properly detached.
Properly register the Network::ConnectionState enum
The Q_ENUMS declaration was, uhm, creative, and additionally Q_ENUMS are automatically
registered with the QMetaType system, so no need for registering them again. Actually,
while this somehow works in Qt4, Qt5 barfs with an assert.
160c2f0...
by
Marcus Eggenberger <email address hidden>
Only show notifications for buffers, which are part of a chatlist currently in use
First of all, takeFirst() does not work on a const ref *cough*. And second, we should
not reorder the param in the message like we did before purely for prettyness;
some servers seem to send non-standard strings that then look awkward. Instead,
let's just add a colon after the first param.