Fix Grayscale depth reduction with tRNS (#470)
This commit is contained in:
parent
19be14021b
commit
54ee621e41
4 changed files with 47 additions and 1 deletions
|
|
@ -144,6 +144,28 @@ pub fn reduce_bit_depth_8_or_less(png: &PngImage, mut minimum_bits: usize) -> Op
|
|||
}
|
||||
}
|
||||
|
||||
// If the image is grayscale we also need to reduce the transparency pixel
|
||||
let mut transparency_pixel = png
|
||||
.transparency_pixel
|
||||
.clone()
|
||||
.filter(|t| png.ihdr.color_type == ColorType::Grayscale && t.len() >= 2);
|
||||
if let Some(trans) = transparency_pixel {
|
||||
let reduced_trans = trans[1] >> (bit_depth - minimum_bits);
|
||||
// Verify the reduction is valid by restoring back to original bit depth
|
||||
let mut check = reduced_trans;
|
||||
let mut bits = minimum_bits;
|
||||
while bits < bit_depth {
|
||||
check = check << bits | check;
|
||||
bits <<= 1;
|
||||
}
|
||||
if trans[0] == 0 && trans[1] == check {
|
||||
transparency_pixel = Some(vec![0, reduced_trans]);
|
||||
} else {
|
||||
// The transparency doesn't fit the new bit depth and is therefore unused - set it to None
|
||||
transparency_pixel = None;
|
||||
}
|
||||
}
|
||||
|
||||
Some(PngImage {
|
||||
data: reduced.as_raw_slice().to_vec(),
|
||||
ihdr: IhdrData {
|
||||
|
|
@ -152,6 +174,6 @@ pub fn reduce_bit_depth_8_or_less(png: &PngImage, mut minimum_bits: usize) -> Op
|
|||
},
|
||||
aux_headers: png.aux_headers.clone(),
|
||||
palette: png.palette.clone(),
|
||||
transparency_pixel: png.transparency_pixel.clone(),
|
||||
transparency_pixel,
|
||||
})
|
||||
}
|
||||
|
|
|
|||
BIN
tests/files/issue-426-01.png
Normal file
BIN
tests/files/issue-426-01.png
Normal file
Binary file not shown.
|
After Width: | Height: | Size: 13 KiB |
BIN
tests/files/issue-426-02.png
Normal file
BIN
tests/files/issue-426-02.png
Normal file
Binary file not shown.
|
After Width: | Height: | Size: 13 KiB |
|
|
@ -642,3 +642,27 @@ fn issue_195() {
|
|||
BitDepth::Eight,
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn issue_426_01() {
|
||||
test_it_converts(
|
||||
"tests/files/issue-426-01.png",
|
||||
None,
|
||||
ColorType::Grayscale,
|
||||
BitDepth::Eight,
|
||||
ColorType::Grayscale,
|
||||
BitDepth::One,
|
||||
);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn issue_426_02() {
|
||||
test_it_converts(
|
||||
"tests/files/issue-426-02.png",
|
||||
None,
|
||||
ColorType::Grayscale,
|
||||
BitDepth::Eight,
|
||||
ColorType::Grayscale,
|
||||
BitDepth::One,
|
||||
);
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in a new issue