1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QPainter: simplify the check whether the new brush style is already set

Browse Source 
The explicit operator==(QBrush, Qt::BrushStyle) does that already, and
more correctly, as it follows the semantics of brush == QBrush(pattern).

Change-Id: I35ce25fe2e895e441ed7974d35bef762c7c533cc
Reviewed-by: Christian Ehrlicher <ch.ehrlicher@gmx.de>
This commit is contained in:
Volker Hilsheimer 2024-11-27 17:00:26 +01:00
parent 2ad79c626d
commit 17ac447f46
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/gui/painting/qpainter.cpp
View File

@ -3763,9 +3763,7 @@ void QPainter::setBrush(Qt::BrushStyle style)
qWarning("QPainter::setBrush: Painter not active");
return;
}
if (d->state->brush.style() == style &&
(style == Qt::NoBrush
|| (style == Qt::SolidPattern && d->state->brush.color() == QColor(0, 0, 0))))
if (d->state->brush == style)
return;
d->state->brush = QBrush(Qt::black, style);
if (d->extended)