ImageData can track whether they're meant to be loaded into a Texture…

… with a non-sRGB format when gamma correct rendering is enabled. Fixes #1556. * Add ImageData/CompressedImageData:setLinear and ImageData/CompressedImageData:isLinear. The flag is used as a hint when loading a texture from the data to determine if the format should not be treated as sRGB-encoded. The 'linear' flag in newImage overrides this. * love.graphics.readbackTexture automatically sets the linear flag on ImageData it returns when the texture's format is linear. This allows an ImageData generated via readback to be fed back into a new Texture and the format will match the original Canvas. * Also clean up some image decoding code.
love2d · Oct 14, 2023 · 8c13943 · 8c13943
1 parent bd55100
commit 8c13943
Show file tree

Hide file tree

Showing 26 changed files with 149 additions and 105 deletions.
diff --git a/src/common/pixelformat.cpp b/src/common/pixelformat.cpp
@@ -364,11 +364,8 @@ PixelFormat getSRGBPixelFormat(PixelFormat format)
  case PIXELFORMAT_ASTC_10x10_UNORM: return PIXELFORMAT_ASTC_10x10_sRGB;
  case PIXELFORMAT_ASTC_12x10_UNORM: return PIXELFORMAT_ASTC_12x10_sRGB;
  case PIXELFORMAT_ASTC_12x12_UNORM: return PIXELFORMAT_ASTC_12x12_sRGB;
- default:
- break;
+ default: return format;
  }
-
- return format;
 }
 
 PixelFormat getLinearPixelFormat(PixelFormat format)
@@ -398,11 +395,8 @@ PixelFormat getLinearPixelFormat(PixelFormat format)
  case PIXELFORMAT_ASTC_10x10_sRGB: return PIXELFORMAT_ASTC_10x10_UNORM;
  case PIXELFORMAT_ASTC_12x10_sRGB: return PIXELFORMAT_ASTC_12x10_UNORM;
  case PIXELFORMAT_ASTC_12x12_sRGB: return PIXELFORMAT_ASTC_12x12_UNORM;
- default:
- break;
+ default: return format;
  }
-
- return format;
 }
 
 size_t getPixelFormatBlockSize(PixelFormat format)

diff --git a/src/modules/graphics/GraphicsReadback.cpp b/src/modules/graphics/GraphicsReadback.cpp
@@ -81,6 +81,7 @@ GraphicsReadback::GraphicsReadback(Graphics *gfx, ReadbackMethod method, Texture
  }
 
  textureFormat = getLinearPixelFormat(texture->getPixelFormat());
+ isFormatLinear = isGammaCorrect() && !isPixelFormatSRGB(texture->getPixelFormat());
 
  if (!image::ImageData::validPixelFormat(textureFormat))
  {
@@ -96,7 +97,7 @@ GraphicsReadback::GraphicsReadback(Graphics *gfx, ReadbackMethod method, Texture
  if (isRT && !caps.features[Graphics::FEATURE_COPY_RENDER_TARGET_TO_BUFFER])
  throw love::Exception("readbackTextureAsync is not supported on this system.");
  else if (!isRT && !caps.features[Graphics::FEATURE_COPY_TEXTURE_TO_BUFFER])
- throw love::Exception("readbackTextureAsync a with non-render-target textures is not supported on this system.");
+ throw love::Exception("readbackTextureAsync with a non-render-target texture is not supported on this system.");
  }
  else
  {
@@ -158,6 +159,7 @@ void *GraphicsReadback::prepareReadbackDest(size_t size)
  throw love::Exception("The love.image module must be loaded for readbackTexture.");
 
  imageData.set(module->newImageData(rect.w, rect.h, textureFormat, nullptr), Acquire::NORETAIN);
+ imageData->setLinear(isFormatLinear);
  return imageData->getData();
  }
  }

diff --git a/src/modules/graphics/GraphicsReadback.h b/src/modules/graphics/GraphicsReadback.h
@@ -103,6 +103,7 @@ class GraphicsReadback : public love::Object
  StrongRef<love::image::ImageData> imageData;
  Rect rect = {};
  PixelFormat textureFormat = PIXELFORMAT_UNKNOWN;
+ bool isFormatLinear = false;
  int imageDataX = 0;
  int imageDataY = 0;
 

diff --git a/src/modules/graphics/Texture.cpp b/src/modules/graphics/Texture.cpp
@@ -203,7 +203,7 @@ Texture::Texture(Graphics *gfx, const Settings &settings, const Slices *slices)
  love::image::ImageDataBase *slice = slices->get(0, 0);
 
  format = slice->getFormat();
- if (isGammaCorrect() && !settings.linear)
+ if (isGammaCorrect() && !slice->isLinear())
  format = getSRGBPixelFormat(format);
 
  pixelWidth = slice->getWidth();
@@ -886,6 +886,7 @@ bool Texture::Slices::validate() const
  int w = firstdata->getWidth();
  int h = firstdata->getHeight();
  PixelFormat format = firstdata->getFormat();
+ bool linear = firstdata->isLinear();
 
  if (textureType == TEXTURE_CUBE && w != h)
  throw love::Exception("Cube textures must have equal widths and heights for each cube face.");
@@ -925,6 +926,9 @@ bool Texture::Slices::validate() const
 
  if (format != slicedata->getFormat())
  throw love::Exception("All texture slices and mipmaps must have the same pixel format.");
+
+ if (linear != slicedata->isLinear())
+ throw love::Exception("All texture slices and mipmaps must have the same linear setting.");
  }
 
  mipw = std::max(mipw / 2, 1);

diff --git a/src/modules/image/CompressedImageData.cpp b/src/modules/image/CompressedImageData.cpp
@@ -30,7 +30,6 @@ love::Type CompressedImageData::type("CompressedImageData", &Data::type);
 
 CompressedImageData::CompressedImageData(const std::list<FormatHandler *> &formats, Data *filedata)
  : format(PIXELFORMAT_UNKNOWN)
- , sRGB(false)
 {
  FormatHandler *parser = nullptr;
 
@@ -46,7 +45,7 @@ CompressedImageData::CompressedImageData(const std::list<FormatHandler *> &forma
  if (parser == nullptr)
  throw love::Exception("Could not parse compressed data: Unknown format.");
 
- memory = parser->parseCompressed(filedata, dataImages, format, sRGB);
+ memory = parser->parseCompressed(filedata, dataImages, format);
 
  if (memory == nullptr)
  throw love::Exception("Could not parse compressed data.");
@@ -56,11 +55,14 @@ CompressedImageData::CompressedImageData(const std::list<FormatHandler *> &forma
 
  if (dataImages.size() == 0 || memory->getSize() == 0)
  throw love::Exception("Could not parse compressed data: No valid data?");
+
+ // This throws away some information the decoder could give us, but we
+ // can't really rely on it I think...
+ format = getLinearPixelFormat(format);
 }
 
 CompressedImageData::CompressedImageData(const CompressedImageData &c)
  : format(c.format)
- , sRGB(c.sRGB)
 {
  memory.set(c.memory->clone(), Acquire::NORETAIN);
 
@@ -134,9 +136,15 @@ PixelFormat CompressedImageData::getFormat() const
  return format;
 }
 
-bool CompressedImageData::isSRGB() const
+void CompressedImageData::setLinear(bool linear)
+{
+ for (auto &slice : dataImages)
+ slice->setLinear(linear);
+}
+
+bool CompressedImageData::isLinear() const
 {
- return sRGB;
+ return dataImages.empty() ? false : dataImages[0]->isLinear();
 }
 
 CompressedSlice *CompressedImageData::getSlice(int slice, int miplevel) const

diff --git a/src/modules/image/CompressedImageData.h b/src/modules/image/CompressedImageData.h
@@ -93,14 +93,14 @@ class CompressedImageData : public Data
  **/
  PixelFormat getFormat() const;
 
- bool isSRGB() const;
+ void setLinear(bool linear);
+ bool isLinear() const;
 
  CompressedSlice *getSlice(int slice, int miplevel) const;
 
 protected:
 
  PixelFormat format;
- bool sRGB;
 
  // Single block of memory containing all of the sub-images.
  StrongRef<ByteData> memory;

diff --git a/src/modules/image/CompressedSlice.cpp b/src/modules/image/CompressedSlice.cpp
@@ -31,7 +31,6 @@ CompressedSlice::CompressedSlice(PixelFormat format, int width, int height, Byte
  , memory(memory)
  , offset(offset)
  , dataSize(size)
- , sRGB(false)
 {
 }
 
@@ -40,7 +39,6 @@ CompressedSlice::CompressedSlice(const CompressedSlice &s)
  , memory(s.memory)
  , offset(s.offset)
  , dataSize(s.dataSize)
- , sRGB(s.sRGB)
 {
 }
 

diff --git a/src/modules/image/CompressedSlice.h b/src/modules/image/CompressedSlice.h
@@ -46,15 +46,13 @@ class CompressedSlice : public ImageDataBase
  CompressedSlice *clone() const override;
  void *getData() const override { return (uint8 *) memory->getData() + offset; }
  size_t getSize() const override { return dataSize; }
- bool isSRGB() const override { return sRGB; }
  size_t getOffset() const { return offset; }
 
 private:
 
  StrongRef<ByteData> memory;
  size_t offset;
  size_t dataSize;
- bool sRGB;
 
 }; // CompressedSlice
 

diff --git a/src/modules/image/FormatHandler.cpp b/src/modules/image/FormatHandler.cpp
@@ -60,7 +60,7 @@ bool FormatHandler::canParseCompressed(Data* /*data*/)
  return false;
 }
 
-StrongRef<ByteData> FormatHandler::parseCompressed(Data* /*filedata*/, std::vector<StrongRef<CompressedSlice>>& /*images*/, PixelFormat& /*format*/, bool& /*sRGB*/)
+StrongRef<ByteData> FormatHandler::parseCompressed(Data* /*filedata*/, std::vector<StrongRef<CompressedSlice>>& /*images*/, PixelFormat& /*format*/)
 {
  throw love::Exception("Compressed image parsing is not implemented for this format backend.");
 }

diff --git a/src/modules/image/FormatHandler.h b/src/modules/image/FormatHandler.h
@@ -106,13 +106,12 @@ class FormatHandler : public love::Object
  * @param[out] images The list of sub-images generated. Byte data is a
  * pointer to the returned data.
  * @param[out] format The format of the Compressed Data.
- * @param[out] sRGB Whether the texture is sRGB-encoded.
  *
  * @return The single block of memory containing the parsed images.
  **/
  virtual StrongRef<ByteData> parseCompressed(Data *filedata,
  std::vector<StrongRef<CompressedSlice>> &images,
- PixelFormat &format, bool &sRGB);
+ PixelFormat &format);
 
  /**
  * Frees raw pixel memory allocated by the format handler.

diff --git a/src/modules/image/ImageData.cpp b/src/modules/image/ImageData.cpp
@@ -152,6 +152,10 @@ void ImageData::decode(Data *data)
  else
  delete[] this->data;
 
+ // This throws away some information the decoder could give us, but we
+ // can't really rely on it I think...
+ decodedimage.format = getLinearPixelFormat(decodedimage.format);
+
  this->width = decodedimage.width;
  this->height = decodedimage.height;
  this->data = decodedimage.data;
@@ -247,11 +251,6 @@ void *ImageData::getData() const
  return data;
 }
 
-bool ImageData::isSRGB() const
-{
- return false;
-}
-
 bool ImageData::inside(int x, int y) const
 {
  return x >= 0 && x < getWidth() && y >= 0 && y < getHeight();

diff --git a/src/modules/image/ImageData.h b/src/modules/image/ImageData.h
@@ -62,7 +62,7 @@ class ImageData : public ImageDataBase
  static love::Type type;
 
  ImageData(Data *data);
- ImageData(int width, int height, PixelFormat format = PIXELFORMAT_RGBA8_UNORM);
+ ImageData(int width, int height, PixelFormat format);
  ImageData(int width, int height, PixelFormat format, void *data, bool own);
  ImageData(const ImageData &c);
  virtual ~ImageData();
@@ -116,7 +116,6 @@ class ImageData : public ImageDataBase
  ImageData *clone() const override;
  void *getData() const override;
  size_t getSize() const override;
- bool isSRGB() const override;
 
  size_t getPixelSize() const;
 

diff --git a/src/modules/image/ImageDataBase.cpp b/src/modules/image/ImageDataBase.cpp
@@ -29,6 +29,7 @@ ImageDataBase::ImageDataBase(PixelFormat format, int width, int height)
  : format(format)
  , width(width)
  , height(height)
+ , linear(false)
 {
 }
 
@@ -47,5 +48,15 @@ int ImageDataBase::getHeight() const
  return height;
 }
 
+void ImageDataBase::setLinear(bool linear)
+{
+ this->linear = linear;
+}
+
+bool ImageDataBase::isLinear() const
+{
+ return linear;
+}
+
 } // image
 } // love
diff --git a/src/modules/image/ImageDataBase.h b/src/modules/image/ImageDataBase.h
@@ -40,7 +40,8 @@ class ImageDataBase : public Data
  int getWidth() const;
  int getHeight() const;
 
- virtual bool isSRGB() const = 0;
+ void setLinear(bool linear);
+ bool isLinear() const;
 
 protected:
 
@@ -50,6 +51,8 @@ class ImageDataBase : public Data
  int width;
  int height;
 
+ bool linear;
+
 }; // ImageDataBase
 
 } // image

diff --git a/src/modules/image/magpie/ASTCHandler.cpp b/src/modules/image/magpie/ASTCHandler.cpp
@@ -105,7 +105,7 @@ bool ASTCHandler::canParseCompressed(Data *data)
  return true;
 }
 
-StrongRef<ByteData> ASTCHandler::parseCompressed(Data *filedata, std::vector<StrongRef<CompressedSlice>> &images, PixelFormat &format, bool &sRGB)
+StrongRef<ByteData> ASTCHandler::parseCompressed(Data *filedata, std::vector<StrongRef<CompressedSlice>> &images, PixelFormat &format)
 {
  if (!canParseCompressed(filedata))
  throw love::Exception("Could not decode compressed data (not an .astc file?)");
@@ -138,8 +138,6 @@ StrongRef<ByteData> ASTCHandler::parseCompressed(Data *filedata, std::vector<Str
  images.emplace_back(new CompressedSlice(cformat, sizeX, sizeY, memory, 0, totalsize), Acquire::NORETAIN);
 
  format = cformat;
- sRGB = false;
-
  return memory;
 }
 

diff --git a/src/modules/image/magpie/ASTCHandler.h b/src/modules/image/magpie/ASTCHandler.h
@@ -45,7 +45,7 @@ class ASTCHandler : public FormatHandler
 
  StrongRef<ByteData> parseCompressed(Data *filedata,
  std::vector<StrongRef<CompressedSlice>> &images,
- PixelFormat &format, bool &sRGB) override;
+ PixelFormat &format) override;
 
 }; // ASTCHandler