Bugfixing: Spot color conversion from DeviceN to process color space

This commit is contained in:
Jakub Melka 2021-03-18 20:09:07 +01:00
parent f588dc91dc
commit e9bff58c06
2 changed files with 43 additions and 4 deletions

View File

@ -211,6 +211,23 @@ PDFFloatBitmap PDFFloatBitmap::extractSpotChannel(uint8_t channel) const
return result; return result;
} }
void PDFFloatBitmap::copyChannel(const PDFFloatBitmap& sourceBitmap, uint8_t channelFrom, uint8_t channelTo)
{
Q_ASSERT(getWidth() == sourceBitmap.getWidth());
Q_ASSERT(getHeight() == sourceBitmap.getHeight());
for (size_t x = 0; x < getWidth(); ++x)
{
for (size_t y = 0; y < getHeight(); ++y)
{
PDFConstColorBuffer sourceProcessColorBuffer = sourceBitmap.getPixel(x, y);
PDFColorBuffer targetProcessColorBuffer = getPixel(x, y);
targetProcessColorBuffer[channelTo] = sourceProcessColorBuffer[channelFrom];
}
}
}
PDFFloatBitmap PDFFloatBitmap::resize(size_t width, size_t height, Qt::TransformationMode mode) const PDFFloatBitmap PDFFloatBitmap::resize(size_t width, size_t height, Qt::TransformationMode mode) const
{ {
if (width == 0 || height == 0) if (width == 0 || height == 0)
@ -410,8 +427,8 @@ void PDFFloatBitmap::blend(const PDFFloatBitmap& source,
else else
{ {
// Color channel is active, but select source color only, if it is nonzero // Color channel is active, but select source color only, if it is nonzero
return pixelFormat.hasSpotColorsSubtractive() ? BlendMode::Overprint_SelectNonOneSourceOrBackdrop return pixelFormat.hasProcessColorsSubtractive() ? BlendMode::Overprint_SelectNonOneSourceOrBackdrop
: BlendMode::Overprint_SelectNonZeroSourceOrBackdrop; : BlendMode::Overprint_SelectNonZeroSourceOrBackdrop;
} }
} }
@ -1093,8 +1110,6 @@ void PDFTransparencyRenderer::collapseSpotColorsToDeviceColors(PDFFloatBitmapWit
for (uint8_t i = spotColorIndexStart; i < spotColorIndexEnd; ++i) for (uint8_t i = spotColorIndexStart; i < spotColorIndexEnd; ++i)
{ {
// Collapse spot color // Collapse spot color
PDFFloatBitmap spotColorBitmap = bitmap.extractSpotChannel(i);
const PDFInkMapper::ColorInfo* spotColor = m_inkMapper->getActiveSpotColor(i - spotColorIndexStart); const PDFInkMapper::ColorInfo* spotColor = m_inkMapper->getActiveSpotColor(i - spotColorIndexStart);
Q_ASSERT(spotColor); Q_ASSERT(spotColor);
@ -1102,6 +1117,8 @@ void PDFTransparencyRenderer::collapseSpotColorsToDeviceColors(PDFFloatBitmapWit
{ {
case PDFAbstractColorSpace::ColorSpace::Separation: case PDFAbstractColorSpace::ColorSpace::Separation:
{ {
PDFFloatBitmap spotColorBitmap = bitmap.extractSpotChannel(i);
PDFFloatBitmap processColorBitmap(spotColorBitmap.getWidth(), spotColorBitmap.getHeight(), PDFPixelFormat::createFormat(pixelFormat.getProcessColorChannelCount(), 0, false, pixelFormat.hasProcessColorsSubtractive(), false)); PDFFloatBitmap processColorBitmap(spotColorBitmap.getWidth(), spotColorBitmap.getHeight(), PDFPixelFormat::createFormat(pixelFormat.getProcessColorChannelCount(), 0, false, pixelFormat.hasProcessColorsSubtractive(), false));
if (!PDFAbstractColorSpace::transform(spotColor->colorSpace.data(), bitmap.getColorSpace().data(), getCMS(), getGraphicState()->getRenderingIntent(), spotColorBitmap.getPixels(), processColorBitmap.getPixels(), this)) if (!PDFAbstractColorSpace::transform(spotColor->colorSpace.data(), bitmap.getColorSpace().data(), getCMS(), getGraphicState()->getRenderingIntent(), spotColorBitmap.getPixels(), processColorBitmap.getPixels(), this))
{ {
@ -1112,6 +1129,22 @@ void PDFTransparencyRenderer::collapseSpotColorsToDeviceColors(PDFFloatBitmapWit
break; break;
} }
case PDFAbstractColorSpace::ColorSpace::DeviceN:
{
PDFFloatBitmap deviceNBitmap(bitmap.getWidth(), bitmap.getHeight(), PDFPixelFormat::createFormat(uint8_t(spotColor->colorSpace->getColorComponentCount()), 0, false, true, false));
PDFFloatBitmap processColorBitmap(bitmap.getWidth(), bitmap.getHeight(), PDFPixelFormat::createFormat(pixelFormat.getProcessColorChannelCount(), 0, false, pixelFormat.hasProcessColorsSubtractive(), false));
deviceNBitmap.copyChannel(bitmap, i, spotColor->colorSpaceIndex);
if (!PDFAbstractColorSpace::transform(spotColor->colorSpace.data(), bitmap.getColorSpace().data(), getCMS(), getGraphicState()->getRenderingIntent(), deviceNBitmap.getPixels(), processColorBitmap.getPixels(), this))
{
reportRenderError(RenderErrorType::Error, PDFTranslationContext::tr("Transformation of spot color to blend color space failed."));
}
bitmap.blendConvertedSpots(processColorBitmap);
break;
}
default: default:
reportRenderError(RenderErrorType::Error, PDFTranslationContext::tr("Transformation of spot color to blend color space failed.")); reportRenderError(RenderErrorType::Error, PDFTranslationContext::tr("Transformation of spot color to blend color space failed."));
break; break;

View File

@ -227,6 +227,12 @@ public:
/// \param channel Channel /// \param channel Channel
PDFFloatBitmap extractSpotChannel(uint8_t channel) const; PDFFloatBitmap extractSpotChannel(uint8_t channel) const;
/// Copies channel from source bitmap to target channel in this bitmap
/// \param sourceBitmap Source bitmap
/// \param channelFrom Source channel
/// \param channelTo Target channel
void copyChannel(const PDFFloatBitmap& sourceBitmap, uint8_t channelFrom, uint8_t channelTo);
/// Resize the bitmap using given transformation mode. Fast transformation mode /// Resize the bitmap using given transformation mode. Fast transformation mode
/// uses nearest neighbour mapping, smooth transformation mode uses weighted /// uses nearest neighbour mapping, smooth transformation mode uses weighted
/// averaging algorithm. /// averaging algorithm.