Modernize blurhashprovider implementation
Might fix #844 See https://code.qt.io/cgit/qt/qtdeclarative.git/commit/examples/quick/imageresponseprovider/imageresponseprovider.cpp?h=5.15&id=b1f238568214e6587b829d6695677e55a99b1d40 for context.
This commit is contained in:
parent
b9a4f369e1
commit
d424145ee4
2 changed files with 45 additions and 20 deletions
|
|
@ -11,17 +11,16 @@
|
|||
#include "blurhash.hpp"
|
||||
|
||||
void
|
||||
BlurhashResponse::run()
|
||||
BlurhashRunnable::run()
|
||||
{
|
||||
if (m_requestedSize.width() < 0 || m_requestedSize.height() < 0) {
|
||||
m_error = QStringLiteral("Blurhash needs size request");
|
||||
emit finished();
|
||||
emit error("Blurhash needs size request");
|
||||
return;
|
||||
}
|
||||
if (m_requestedSize.width() == 0 || m_requestedSize.height() == 0) {
|
||||
m_image = QImage(m_requestedSize, QImage::Format_RGB32);
|
||||
m_image.fill(QColor(0, 0, 0));
|
||||
emit finished();
|
||||
auto image = QImage(m_requestedSize, QImage::Format_RGB32);
|
||||
image.fill(QColor(0, 0, 0));
|
||||
emit done(image);
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
@ -29,8 +28,7 @@ BlurhashResponse::run()
|
|||
m_requestedSize.width(),
|
||||
m_requestedSize.height());
|
||||
if (decoded.image.empty()) {
|
||||
m_error = QStringLiteral("Failed decode!");
|
||||
emit finished();
|
||||
emit error(QStringLiteral("Failed decode!"));
|
||||
return;
|
||||
}
|
||||
|
||||
|
|
@ -40,6 +38,5 @@ BlurhashResponse::run()
|
|||
(int)decoded.width * 3,
|
||||
QImage::Format_RGB888);
|
||||
|
||||
m_image = image.copy();
|
||||
emit finished();
|
||||
emit done(std::move(image));
|
||||
}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue