Symbols used for marks can be used in different sizes, e.g. depending of the
line height, to which the icon border adapts, or for the context menu on
actions to toggle those marks. Being limited to set a single pixmap as
symbol for a mark results can result in badly scaled symbols being
displayed.
Switching to QIcon as dynamic pixmap provider for markers improves this.
For backward compatibility QIcon & QPixmaps are converted into each other in
case APIs are used mixed.
Currently this is WIP as KDevelop as user of the mark interfaces partially
relies on providing a single pixmap only, and while taking the pixmaps from
QIcons does some QIcon::Mode-based rendering (for breakpoints, marking
disabled, reached, pending or normal ones) or color-tinting (marking line
removal/addition in patch display with positive & negative colors) before
passing the pixmaps on. I am not yet sure whether to
a) turning all those processings into explisit separate dedicatd icons
(my favourite)
b) adding some additional rendering flags to marks
is the best approach.
If going for a), this patch would be final as is.