[Scummvm-devel] png
D G Turner
d.g.turner at ntlworld.com
Wed Jul 25 00:10:41 CEST 2012
Johannes,
Not to have an argument pointlessly... but I don't think that was the
conclusion of the discussion.
Most of the thread indicated that (apart from the interlacing
support, which has now been added), our internal PNG decoder is
complete for our needs, so throwing it away and switching to an
external library seems like alot of work for little improvement.
I don't think we should add further library dependencies, unless we
have very good reasons. Partly as they are generally not tested
over the variety of platforms we target.
Also, the code required to "shim" the library to the image decoder
API may result in worse overall performance, thus defeating the
object of the optimised library speed improvement.
If the Core team does decide to press ahead with this anyway, I would
suggest that a solution is implemented where the internal PNG decoder
is retained as a fallback for builds when libpng is not present, thus
ensuring that this is not a mandatory build dependency.
Thanks,
David Turner
P.S libjpeg has been a slightly different issue as the IDCT does
benefit from hardware acceleration support... but this is somewhat
pointless as this is mainly available on high-end/desktop targets
which are not the problematic case.
On 07/24/12 22:53, Johannes Schickel wrote:
> On 07/18/2012 04:57 PM, Willem Jan Palenstijn wrote:
>> Given the missing features (and/or bugs) in our current PNG decoding
>> [ https://github.com/scummvm/scummvm/pull/258 ], should we revisit our choice
>> not to use libpng? That seems to me to be the easiest way to get all required
>> features in a well-tested and supported form.
>>
>
> As far as I can tell, most people agree it's fine to switch to libpng.
> So who will look into implementing this change?
>
> // Johannes
More information about the Scummvm-devel
mailing list