The only missing thing in our custom PNG code (based on LodePNG) was support for interlaced PNGs, which are rarely used in games (apart from some WME indie games). This is what that pull request refers to. The rest of the code does function...<span></span><div>
<br></div><div>If you do wish to drop the custop PNG decoder, shouldn't we do the same with the JPEG decoder and use libjpeg? IIRC, that one also had missing features.</div><div><br></div><div>Regards</div><div>Filippos<br>
<br>On Wednesday, July 18, 2012, Eugene Sandulenko  wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 18 July 2012 17:57, Willem Jan Palenstijn <<a href="javascript:;" onclick="_e(event, 'cvml', 'wjp@usecode.org')">wjp@usecode.org</a>> wrote:<br>

> Given the missing features (and/or bugs) in our current PNG decoding<br>
> [ <a href="https://github.com/scummvm/scummvm/pull/258" target="_blank">https://github.com/scummvm/scummvm/pull/258</a> ], should we revisit our choice<br>
> not to use libpng? That seems to me to be the easiest way to get all required<br>
> features in a well-tested and supported form.<br>
I would say it's better to switch to libpng now.<br>
<br>
In the past there were couple of issues connected with it. First was<br>
the code size, but since NDS port seems to be dead now, it is not an<br>
issue. Another thing is upgrading all of the platforms on the buildbot<br>
and upgrading all of the ports.<br>
<br>
Which brings to my mind an issue. I wonder whether all of our ports<br>
incorporated freetype2? Seem not.<br>
<br>
<br>
Eugene<br>
<br>
------------------------------------------------------------------------------<br>
Live Security Virtual Conference<br>
Exclusive live event will cover all the ways today's security and<br>
threat landscape has changed and how IT managers can respond. Discussions<br>
will include endpoint security, mobile security and the latest in malware<br>
threats. <a href="http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/" target="_blank">http://www.accelacomm.com/jaw/sfrnl04242012/114/50122263/</a><br>
_______________________________________________<br>
Scummvm-devel mailing list<br>
<a href="javascript:;" onclick="_e(event, 'cvml', 'Scummvm-devel@lists.sourceforge.net')">Scummvm-devel@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/scummvm-devel" target="_blank">https://lists.sourceforge.net/lists/listinfo/scummvm-devel</a><br>
</blockquote></div><br><br>-- <br>"Experience is the name every one gives to their mistakes" - Oscar Wilde <br>