<div dir="ltr"><div id="gmail-:ofww" class="gmail-Am gmail-aO9 gmail-Al editable gmail-LW-avf gmail-tS-tW gmail-tS-tY" aria-label="Message Body" role="textbox" aria-multiline="true" tabindex="1" style="direction:ltr;min-height:85px" aria-controls=":rj4s">On further testing:<div><br></div><div>up to 4.4: fine<br></div><div><a href="https://squeak.js.org/run/#zip=https://files.squeak.org/4.4/Squeak4.4-12327.zip" target="_blank">https://squeak.js.org/run/#zip=https://files.squeak.org/4.4/Squeak4.4-12327.zip</a><br><br>since 4.5: broken<br></div><div><a href="https://squeak.js.org/run/#zip=https://files.squeak.org/4.5/Squeak4.5-13680.zip" target="_blank">https://squeak.js.org/run/#zip=https://files.squeak.org/4.5/Squeak4.5-13680.zip</a><br></div><div><br></div><div>Filing 4.4's PNGReadWriter into 4.5 fixes the yellowing issue.</div><div><br></div><div>Just filing in 4.4's PNGReadWriter>>copyPixelsRGB: also fixes the yellowing issue.</div><div><br></div><div>Attaching ...</div><div><br></div><div>(I realize this may be entirely unrelated to the issue you are seeing)</div><div><br></div><div>Vanessa</div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Feb 25, 2023 at 3:37 PM tim Rowledge <<a href="mailto:tim@rowledge.org">tim@rowledge.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
<br>
> On 2023-02-25, at 2:41 PM, Vanessa Freudenberg <<a href="mailto:vanessa@codefrau.net" target="_blank">vanessa@codefrau.net</a>> wrote:<br>
> <br>
> Ohhh would you look at that ... works perfectly in a 2.8 image:<br>
<br>
<br>
Oh, right, so we've now written advanced new technology bugs. Lovely! These problem files are from CCTV cameras so who knows how well they obey any standards. <br>
<br>
The current PNGReadWriter has a debug output to the Transcript that tells me the image has -<br>
form = Form(252x172x32)<br>
colorType = 2<br>
interlaceMethod = 1<br>
filters = {137->1 . 129->3 . 54->2 . 3->4}<br>
<br>
whereas all the PNG's I can find that load 'properly' seem to have colorType = 6, interlace method = 0 and a bit oddly, appear to repeat the debug output as if it is processed twice? Hmm, shomething wrong offisher? Still, a different interlace code might be a good clue given the image actually *looks* like interlace-screwed.<br>
<br>
Oh, so much for that theory, I found a demo png with colorType = 3 and interlace 1. Poo.<br>
<br>
tim<br>
--<br>
tim Rowledge; <a href="mailto:tim@rowledge.org" target="_blank">tim@rowledge.org</a>; <a href="http://www.rowledge.org/tim" rel="noreferrer" target="_blank">http://www.rowledge.org/tim</a><br>
Useful Latin Phrases:- Mellita, domi adsum. = Honey, I'm home.<br>
<br>
<br>
<br>
</blockquote></div>