Not sure what operator+ overload has been used exactly for the int,
in any case it does not work as intended and needs e.g. an explicit
QString::number() invocation.
Also start with number 2 for duplicated instances.
mtijink |
KDE Connect |
Not sure what operator+ overload has been used exactly for the int,
in any case it does not work as intended and needs e.g. an explicit
QString::number() invocation.
Also start with number 2 for duplicated instances.
Start multiple instances of an MPRIS player (e.g. Gwenview). Before the
second instance would get labelled with "Name []", with this patch it is
labelled with "Name [2]".
Automatic diff as part of commit; lint not applicable. |
Automatic diff as part of commit; unit tests not applicable. |
While looking as this: perhaps it make sense to also switch to use "<#>" instead of "[#]", like known e.g. from KWin? Is there any standard for such instance label numbering?
I guess <> makes sense. I dont't know if there is a standard for this (maybe ask VDG ), but feel free to go ahead with <>