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

Jean-Baptiste Mardelle <j...@kdenlive.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/mult |https://invent.kde.org/mult
                   |imedia/kdenlive/commit/3f81 |imedia/kdenlive/commit/e0f6
                   |05070aeedcf09adba716f79639f |8346d015180a0ccce2fd740bb35
                   |c63f72f96                   |98f359a4a

--- Comment #5 from Jean-Baptiste Mardelle <j...@kdenlive.org> ---
Git commit e0f68346d015180a0ccce2fd740bb3598f359a4a by Jean-Baptiste Mardelle,
on behalf of Eric Jiang.
Committed on 27/07/2022 at 11:47.
Pushed by mardelle into branch 'release/22.08'.

Change vectorscope to use QImage::pixel()

Vectorscope tried to directly reinterpret the input QImage as RGB
data, which makes an assumption that the input is RGB. On Windows, the
input QImage is BGR, causing red and blue to be switched in the
Vectorscope.

QImage::pixel() checks QImageData->format and converts the to QRgb
appropriately. Casual benchmarking seems to show that using pixel() is
about 5% slower for each vectorscope calculation in the no-conversion
case (e.g. Linux).

M  +13   -12   src/scopes/colorscopes/vectorscopegenerator.cpp

https://invent.kde.org/multimedia/kdenlive/commit/e0f68346d015180a0ccce2fd740bb3598f359a4a

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

Reply via email to