From 9e3690670a283bee24969add194d39fa4531b3af Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vladim=C3=ADr=20Vondru=C5=A1?= Date: Sat, 2 Mar 2019 03:19:08 +0100 Subject: [PATCH] TextureTools: explicitly force operation sequencing in the test. On GCC 4.8 at least, release() gets called on the image before size(), causing size() to be zero. --- src/Magnum/TextureTools/Test/DistanceFieldGLTest.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/Magnum/TextureTools/Test/DistanceFieldGLTest.cpp b/src/Magnum/TextureTools/Test/DistanceFieldGLTest.cpp index 831b9b423..b3c7f94dd 100644 --- a/src/Magnum/TextureTools/Test/DistanceFieldGLTest.cpp +++ b/src/Magnum/TextureTools/Test/DistanceFieldGLTest.cpp @@ -194,7 +194,8 @@ void DistanceFieldGLTest::test() { /* In some cases actualOutputImage might have GL-specific format, reinterpret as R8Unorm for the comparison to work */ if(actualOutputImage->format() == pixelFormatWrap(GL::PixelFormat::Red)) { - actualOutputImage = Image2D{actualOutputImage->storage(), PixelFormat::R8Unorm, actualOutputImage->size(), actualOutputImage->release()}; + const Vector2i imageSize = actualOutputImage->size(); + actualOutputImage = Image2D{actualOutputImage->storage(), PixelFormat::R8Unorm, imageSize, actualOutputImage->release()}; } #endif