Hi Hilaire, On Pharo 7 build 993 it works on both 32 and 64 bits (macOS): The patch (#normalize calls here and there) seems to have been integrated as well your test case (#testNewFromSampleBytes), with your name in the comments ! Sven
On 31 May 2018, at 13:23, Hilaire <hilaire@drgeo.eu <mailto:hilaire@drgeo.eu>> wrote:
Ah, indeed testing on latest P7, this code works.
Oh, but wait PNG reader seems to break now on another part... See screenshot.
Oh, it looks like the problem we got almost one years ago... where I suggested code snipped for test inclusion[1] and it now fails again (was not failing on my previous P7 image)... As far as I can see I did not find a test based on it in the Pharo image I tested on.
At the time it was failing on 64bits, and now it fails on the 32bits I tried it.
I guess some recent changes on P7 break something. It will be nice to add a test based on the example I provided at [1].
Hilaire
[1] http://forum.world.st/Regression-with-PNGReaderWriter-in-P6-td4951796.html <http://forum.world.st/Regression-with-PNGReaderWriter-in-P6-td4951796.html>
Le 31/05/2018 à 12:20, Sven Van Caekenberghe a écrit :
Hmm, works for me:
-- Dr. Geo http://drgeo.eu <http://drgeo.eu/>
<brokenPNG.png>
