On 27 May 2011 01:46, Geoffrey Hutchison <ge...@geoffhutchison.net> wrote: >> I guess that adding an extra dependency isn't on the cards for 2.3.1, >> but I just thought I'd bring it up in case... > > I'm willing to discuss it. It's basically an optional build dependency, > correct? > > 1) There's no modification to the core. > 2) It doesn't affect libopenbabel, only the depiction module. > > One concern would be if PNG can round-trip with embedded metadata.
It would be optional, for sure. And it doesn't change the public API. I'm not sure what you mean by a depiction module. I think it either has to go into libopenbabel, or else be cut+pasted into the top of pngformat.cpp, unless someone else has an alternative suggestion. I forgot about the existing PNG support. Yes, I guess it would have to be merged with this. Roundtripping should be no problem if we include a mol file in there, for example. - Noel ------------------------------------------------------------------------------ vRanger cuts backup time in half-while increasing security. With the market-leading solution for virtual backup and recovery, you get blazing-fast, flexible, and affordable data protection. Download your free trial now. http://p.sf.net/sfu/quest-d2dcopy1 _______________________________________________ OpenBabel-Devel mailing list OpenBabel-Devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/openbabel-devel