Differential D11630 Diff 30672 lib/documentview/rasterimageview.cpp

Changeset View

Standalone View

View Options

lib/documentview/rasterimageview.cpp

Show First 20 Lines • Show All 158 Lines • ▼ Show 20 Line(s)143 {
159 } 159​ }
160 qSwap(mAlternateBuffer, mCurrentBuffer); 160​ qSwap(mAlternateBuffer, mCurrentBuffer);
161 161​
162 mAlternateBuffer = QPixmap(); 162​ mAlternateBuffer = QPixmap();
163 } 163​ }
164 164​
165 void drawAlphaBackground(QPainter* painter, const QRect& viewportRect, const QPoint& zoomedImageTopLeft, QPixmap texture) 165​ void drawAlphaBackground(QPainter* painter, const QRect& viewportRect, const QPoint& zoomedImageTopLeft, QPixmap texture)
166 { 166​ {
167 if (mAlphaBackgroundMode == AbstractImageView::AlphaBackgroundCheckBoard) { 167​ switch (mAlphaBackgroundMode) {
168​ case (AbstractImageView::AlphaBackgroundCheckBoard): {
ngrahamUnsubmitted
Not Done

Style: In general, case/switch statements in KDE code don't use braces.

ngraham: Style: In general, case/switch statements in KDE code don't use braces.
huoniAuthorUnsubmitted
Not Done

textureOffset is defined in the first case. To take away the braces, I'd need to move the definition to before the switch, even though it's only used in the first case. Would that still be preferable?
Perhaps taking away the braces for the cases that don't need them, leaving them for the first case - that way it's less a style decision and more a function decision.

huoni: `textureOffset` is defined in the first case. To take away the braces, I'd need to move the…
zzagUnsubmitted
Not Done

textureOffset is used only once so there is no need to declare a variable.

Also, cases don't need parentheses.

zzag: textureOffset is used only once so there is no need to declare a variable. Also, `case`s don't…
huoniAuthorUnsubmitted
Not Done

I removed the blocks apart from the first one, but changed the opening brace to be on its own line.
This is more like an explicit block used for restricting scope which is used in other parts of the file.

huoni: I removed the blocks apart from the first one, but changed the opening brace to be on its own…
rkflxUnsubmitted
Not Done

Good call, makes sense.

rkflx: Good call, makes sense.
168 QPoint textureOffset( 169​ QPoint textureOffset(
169 zoomedImageTopLeft.x() % texture.width(), 170​ zoomedImageTopLeft.x() % texture.width(),
170 zoomedImageTopLeft.y() % texture.height() 171​ zoomedImageTopLeft.y() % texture.height()
171 ); 172​ );
172 painter->drawTiledPixmap( 173​ painter->drawTiledPixmap(
173 viewportRect, 174​ viewportRect,
174 texture, 175​ texture,
175 textureOffset); 176​ textureOffset);
176 } else { 177​ break;
178​ }
179​ case (AbstractImageView::AlphaBackgroundSolid): {
177 painter->fillRect(viewportRect, mAlphaBackgroundColor); 180​ painter->fillRect(viewportRect, mAlphaBackgroundColor);
181​ break;
182​ }
183​ case (AbstractImageView::AlphaBackgroundNone): {
184​ painter->fillRect(viewportRect, Qt::transparent);
rkflxUnsubmitted
Done

I'd put this as the first case in the switch, to mirror the enum as well as the UI.

rkflx: I'd put this as the first `case` in the `switch`, to mirror the `enum` as well as the UI.
185​ break;
186​ }
187​ default: {
188​ Q_ASSERT(0);
189​ }
178 } 190​ }
179 } 191​ }
180}; 192​};
181 193​
182RasterImageView::RasterImageView(QGraphicsItem* parent) 194​RasterImageView::RasterImageView(QGraphicsItem* parent)
183: AbstractImageView(parent) 195​: AbstractImageView(parent)
184, d(new RasterImageViewPrivate) 196​, d(new RasterImageViewPrivate)
185{ 197​{
186 d->q = this; 198​ d->q = this;
187 d->mEmittedCompleted = false; 199​ d->mEmittedCompleted = false;
188 d->mApplyDisplayTransform = true; 200​ d->mApplyDisplayTransform = true;
189 d->mDisplayTransform = 0; 201​ d->mDisplayTransform = 0;
190 202​
191 d->mAlphaBackgroundMode = AlphaBackgroundCheckBoard; 203​ d->mAlphaBackgroundMode = AlphaBackgroundNone;
192 d->mAlphaBackgroundColor = Qt::black; 204​ d->mAlphaBackgroundColor = Qt::black;
193 d->mRenderingIntent = INTENT_PERCEPTUAL; 205​ d->mRenderingIntent = INTENT_PERCEPTUAL;
194 d->mEnlargeSmallerImages = false; 206​ d->mEnlargeSmallerImages = false;
195 207​
196 d->mBufferIsEmpty = true; 208​ d->mBufferIsEmpty = true;
197 d->mScaler = new ImageScaler(this); 209​ d->mScaler = new ImageScaler(this);
198 connect(d->mScaler, &ImageScaler::scaledRect, this, &RasterImageView::updateFromScaler); 210​ connect(d->mScaler, &ImageScaler::scaledRect, this, &RasterImageView::updateFromScaler);
199 211​
▲ Show 20 Lines • Show All 127 Lines • ▼ Show 20 Line(s)332{
327 } 339​ }
328 340​
329 d->resizeBuffer(); 341​ d->resizeBuffer();
330 int viewportLeft = zoomedImageLeft - scrollPos().x(); 342​ int viewportLeft = zoomedImageLeft - scrollPos().x();
331 int viewportTop = zoomedImageTop - scrollPos().y(); 343​ int viewportTop = zoomedImageTop - scrollPos().y();
332 d->mBufferIsEmpty = false; 344​ d->mBufferIsEmpty = false;
333 { 345​ {
334 QPainter painter(&d->mCurrentBuffer); 346​ QPainter painter(&d->mCurrentBuffer);
347​ painter.setCompositionMode(QPainter::CompositionMode_Source);
335 if (document()->hasAlphaChannel()) { 348​ if (document()->hasAlphaChannel()) {
336 d->drawAlphaBackground( 349​ d->drawAlphaBackground(
337 &painter, QRect(viewportLeft, viewportTop, image.width(), image.height()), 350​ &painter, QRect(viewportLeft, viewportTop, image.width(), image.height()),
338 QPoint(zoomedImageLeft, zoomedImageTop), 351​ QPoint(zoomedImageLeft, zoomedImageTop),
339 alphaBackgroundTexture() 352​ alphaBackgroundTexture()
340 ); 353​ );
341 } else { 354​ // This is required so transparent pixels don't replace our background
342 painter.setCompositionMode(QPainter::CompositionMode_Source); 355​ painter.setCompositionMode(QPainter::CompositionMode_SourceOver);
343 } 356​ }
344 painter.drawImage(viewportLeft, viewportTop, image); 357​ painter.drawImage(viewportLeft, viewportTop, image);
345 } 358​ }
346 update(); 359​ update();
347 360​
348 if (!d->mEmittedCompleted) { 361​ if (!d->mEmittedCompleted) {
349 d->mEmittedCompleted = true; 362​ d->mEmittedCompleted = true;
350 completed(); 363​ completed();
Show All 23 Lines
374{ 387​{
375 QPointF delta = scrollPos() - oldPos; 388​ QPointF delta = scrollPos() - oldPos;
376 389​
377 // Scroll existing 390​ // Scroll existing
378 { 391​ {
379 if (d->mAlternateBuffer.size() != d->mCurrentBuffer.size()) { 392​ if (d->mAlternateBuffer.size() != d->mCurrentBuffer.size()) {
380 d->mAlternateBuffer = QPixmap(d->mCurrentBuffer.size()); 393​ d->mAlternateBuffer = QPixmap(d->mCurrentBuffer.size());
381 } 394​ }
395​ d->mAlternateBuffer.fill(Qt::transparent);
382 QPainter painter(&d->mAlternateBuffer); 396​ QPainter painter(&d->mAlternateBuffer);
383 painter.drawPixmap(-delta, d->mCurrentBuffer); 397​ painter.drawPixmap(-delta, d->mCurrentBuffer);
384 } 398​ }
385 qSwap(d->mCurrentBuffer, d->mAlternateBuffer); 399​ qSwap(d->mCurrentBuffer, d->mAlternateBuffer);
386 ​
rkflxUnsubmitted
Done

Accidental whitespace removal.

rkflx: Accidental whitespace removal.
387 // Scale missing parts 400​ // Scale missing parts
388 QRegion bufferRegion = QRegion(d->mCurrentBuffer.rect().translated(scrollPos().toPoint())); 401​ QRegion bufferRegion = QRegion(d->mCurrentBuffer.rect().translated(scrollPos().toPoint()));
389 QRegion updateRegion = bufferRegion - bufferRegion.translated(-delta.toPoint()); 402​ QRegion updateRegion = bufferRegion - bufferRegion.translated(-delta.toPoint());
390 updateBuffer(updateRegion); 403​ updateBuffer(updateRegion);
391 update(); 404​ update();
392} 405​}
393 406​
394void RasterImageView::paint(QPainter* painter, const QStyleOptionGraphicsItem* /*option*/, QWidget* /*widget*/) 407​void RasterImageView::paint(QPainter* painter, const QStyleOptionGraphicsItem* /*option*/, QWidget* /*widget*/)
▲ Show 20 Lines • Show All 160 Lines • Show Last 20 Lines