https://bugs.kde.org/show_bug.cgi?id=430942

--- Comment #15 from brainpower <brainpo...@mailbox.org> ---
Found the issue:

If I add a few qDebug:

--- a/src/QuickEditor/QuickEditor.cpp
+++ b/src/QuickEditor/QuickEditor.cpp
@@ -116,8 +116,12 @@ QuickEditor::QuickEditor(const QMap<ComparableQPoint,
QImage> &images, KWayland:
         input.insert(pos, QPair<qreal, QSize>(screenImage.width() /
static_cast<qreal>(screen->size().width()), screenImage.size()));
     }
     const auto pointsTranslationMap = computeCoordinatesAfterScaling(input);
+    qDebug() << "pointsTranslationMap:       " << pointsTranslationMap;
+    qDebug() << "pointsTranslationMap.keys():" << pointsTranslationMap.keys();
     QPainter painter(&mPixmap);
     for (auto it = mImages.constBegin(); it != mImages.constEnd(); it ++) {
+        qDebug() << "it.key() == QPoint(1680,0) ->" << (it.key() ==
ComparableQPoint(QPoint(1680,0)));
+        qDebug() << "it.key():" << it.key() << "\t translated:" <<
pointsTranslationMap.value(it.key());
         painter.drawImage(pointsTranslationMap.value(it.key()), it.value());
     }
     painter.end();


I get:

% build/bin/spectacle
pointsTranslationMap:        QMap((QPoint(1680,0),
QPoint(1680,0))(QPoint(0,195), QPoint(0,195))(QPoint(4240,0), QPoint(4240,0)))
pointsTranslationMap.keys(): (QPoint(1680,0), QPoint(0,195), QPoint(4240,0))
it.key() == QPoint(1680,0) -> true
it.key(): QPoint(1680,0)         translated: QPoint(0,0)
it.key() == QPoint(1680,0) -> false
it.key(): QPoint(0,195)          translated: QPoint(0,195)
it.key() == QPoint(1680,0) -> false
it.key(): QPoint(4240,0)         translated: QPoint(4240,0)


But it should be 

it.key(): QPoint(1680,0)         translated: QPoint(1680,0)

instad of

it.key(): QPoint(1680,0)         translated: QPoint(0,0)

I have no idea why the QMap::value() returns the defaultValue here instead of
the value the key should map to...
it seems to exist correctly in the pointsTranslationMap after all...

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to