Merge pull request #4639 from aws-lumberyard-dev/Atom/guthadam/thumbnail_bus_passes_const_qpixmap
Updated thumbnail notification bus to use const QPixmap&
This commit is contained in:
@@ -90,7 +90,7 @@ namespace AzToolsFramework
|
||||
typedef SharedThumbnailKey BusIdType;
|
||||
|
||||
//! notify product thumbnail that the data is ready
|
||||
virtual void ThumbnailRendered(QPixmap& thumbnailImage) = 0;
|
||||
virtual void ThumbnailRendered(const QPixmap& thumbnailImage) = 0;
|
||||
//! notify product thumbnail that the thumbnail failed to render
|
||||
virtual void ThumbnailFailedToRender() = 0;
|
||||
};
|
||||
|
||||
@@ -67,7 +67,7 @@ namespace ImageProcessingAtom
|
||||
m_renderWait.acquire();
|
||||
}
|
||||
|
||||
void ImageThumbnail::ThumbnailRendered(QPixmap& thumbnailImage)
|
||||
void ImageThumbnail::ThumbnailRendered(const QPixmap& thumbnailImage)
|
||||
{
|
||||
m_pixmap = thumbnailImage;
|
||||
m_renderWait.release();
|
||||
|
||||
@@ -34,7 +34,7 @@ namespace ImageProcessingAtom
|
||||
~ImageThumbnail() override;
|
||||
|
||||
//! AzToolsFramework::ThumbnailerRendererNotificationBus::Handler overrides...
|
||||
void ThumbnailRendered(QPixmap& thumbnailImage) override;
|
||||
void ThumbnailRendered(const QPixmap& thumbnailImage) override;
|
||||
void ThumbnailFailedToRender() override;
|
||||
|
||||
protected:
|
||||
|
||||
@@ -54,7 +54,7 @@ namespace AZ
|
||||
AzFramework::AssetCatalogEventBus::Handler::BusDisconnect();
|
||||
}
|
||||
|
||||
void MaterialThumbnail::ThumbnailRendered(QPixmap& thumbnailImage)
|
||||
void MaterialThumbnail::ThumbnailRendered(const QPixmap& thumbnailImage)
|
||||
{
|
||||
m_pixmap = thumbnailImage;
|
||||
m_renderWait.release();
|
||||
|
||||
@@ -36,7 +36,7 @@ namespace AZ
|
||||
~MaterialThumbnail() override;
|
||||
|
||||
//! AzToolsFramework::ThumbnailerRendererNotificationBus::Handler overrides...
|
||||
void ThumbnailRendered(QPixmap& thumbnailImage) override;
|
||||
void ThumbnailRendered(const QPixmap& thumbnailImage) override;
|
||||
void ThumbnailFailedToRender() override;
|
||||
|
||||
protected:
|
||||
|
||||
@@ -55,7 +55,7 @@ namespace AZ
|
||||
AzFramework::AssetCatalogEventBus::Handler::BusDisconnect();
|
||||
}
|
||||
|
||||
void MeshThumbnail::ThumbnailRendered(QPixmap& thumbnailImage)
|
||||
void MeshThumbnail::ThumbnailRendered(const QPixmap& thumbnailImage)
|
||||
{
|
||||
m_pixmap = thumbnailImage;
|
||||
m_renderWait.release();
|
||||
|
||||
@@ -35,7 +35,7 @@ namespace AZ
|
||||
~MeshThumbnail() override;
|
||||
|
||||
//! AzToolsFramework::ThumbnailerRendererNotificationBus::Handler overrides...
|
||||
void ThumbnailRendered(QPixmap& thumbnailImage) override;
|
||||
void ThumbnailRendered(const QPixmap& thumbnailImage) override;
|
||||
void ThumbnailFailedToRender() override;
|
||||
|
||||
protected:
|
||||
|
||||
Reference in New Issue
Block a user