From 155fc950ec2acdc8033e6ddb610c4b97c9111e6f Mon Sep 17 00:00:00 2001 From: Ivailo Monev Date: Sun, 26 Jul 2020 20:11:49 +0000 Subject: [PATCH] replace switch with if statement in QRasterPaintEngine::init() Signed-off-by: Ivailo Monev --- src/gui/painting/qpaintengine_raster.cpp | 12 ++---------- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/src/gui/painting/qpaintengine_raster.cpp b/src/gui/painting/qpaintengine_raster.cpp index 5f47a5fcc..136905f2a 100644 --- a/src/gui/painting/qpaintengine_raster.cpp +++ b/src/gui/painting/qpaintengine_raster.cpp @@ -233,21 +233,13 @@ void QRasterPaintEngine::init() d->mono_surface = false; gccaps &= ~PorterDuff; - QImage::Format format = QImage::Format_Invalid; - - switch (d->device->devType()) { - case QInternal::Pixmap: - qWarning("QRasterPaintEngine: unsupported for pixmaps..."); - break; - case QInternal::Image: - format = d->rasterBuffer->prepare(static_cast(d->device)); - break; - default: + if (Q_UNLIKELY(d->device->devType() != QInternal::Image)) { qWarning("QRasterPaintEngine: unsupported target device %d\n", d->device->devType()); d->device = 0; return; } + QImage::Format format = d->rasterBuffer->prepare(static_cast(d->device)); switch (format) { case QImage::Format_MonoLSB: case QImage::Format_Mono: -- 2.11.0