> > On 8 Nov 2008, at 21:39, Fabien Costantini wrote: > > - I say =F9see,s ok=F9 because I find the lines 580-582 of =20 > > configure.in highly suspect: > > [if test x$enable_localpng =3D xyes; then > > ac_cv_lib_png_png_set_tRNS_to_alpha=3Dno > > fi]) > > I give up on this one. It seems to explicitly disable =20 > png_set_tRNS_to_alpha IF we use the local built-in PNG lib. But the =20 > built-in PNG lib does support png_set_tRNS_to_alpha... Mind you, we =20 > don't actually seem to use the png_png_set_tRNS_to_alpha flag in the =20 > configure script either. > > Maybe it is just a leftover from the days when the built-in PNG lib =20 > was quite an old one? It probably is, I think you should make the change, the only suggestion I would have is if you really have to force these flags in lib mode, please continue to force them in the configuration files (as your patch did), this let the possibility to a look and enhance/fix this configure.in file later for both fltk1 and even fltk2. BTW, if something is wrong after that change as the configure.in file could suggest, we want to know it and fix this nonorthogonality too (if really there is a difference then it would mean we have to update the localpng lib as an example) ; so let's do the change and test that among as many platform as possible ;-)
_______________________________________________ fltk-dev mailing list [email protected] http://lists.easysw.com/mailman/listinfo/fltk-dev
