[OpenJDK 2D-Dev] Question about splashscreen lib

Phil Race philip.race at oracle.com
Wed May 30 17:07:39 UTC 2018


I think PNG_DLL_EXPORT is not intended to be used directly.
I am not sure what problem this causes but it is just the way this 
library is written ...

-phil.

On 05/30/2018 04:16 AM, Adam Farley8 wrote:
> Thanks Philip.
>
> The whole thing with PNG_DLL_EXPORT not being defined in png.h or 
> png.c is still confusing to me
> (pngpriv defines it), especially since png.h ends up using it 
> post-pre-processing, but I don't know if
> that would bother people outside the minority that inspect the 
> pre-processed code.
>
> I'll leave this here as a thought for any committer bothered by this, 
> though I won't be pushing it.
>
> ---- png.h ----
> + #ifndef PNG_DLL_EXPORT
> + #define PNG_DLL_EXPORT
> + #endif
>
> Best Regards
>
> Adam Farley
>
>
> Philip Race <philip.race at oracle.com> wrote on 29/05/2018 14:23:21:
>
> > From: Philip Race <philip.race at oracle.com>
> > To: Adam Farley8 <adam.farley at uk.ibm.com>
> > Cc: 2d-dev <2d-dev at openjdk.java.net>
> > Date: 29/05/2018 14:23
> > Subject: Re: [OpenJDK 2D-Dev] Question about splashscreen lib
> > 
> > So far as I can see it does, just indirectly.
> > png.c includes pngpriv.h which includes png.h
> >
> > Not including the declarations of the exported API in the file that
> > defines them would be a problem waiting to happen.
> >
> > -phil.
> >
> > On 5/29/18, 3:54 AM, Adam Farley8 wrote:
> > Hi All, 
> >
> > Does anyone know why png.c does not include png.h?
> >
> > This tripped me up recently, and I don't understand why png.c would
> > not include png.h.
> >
> > Best Regards
> >
> > Adam Farley
> >
> > Unless stated otherwise above:
> > IBM United Kingdom Limited - Registered in England and Wales with
> > number 741598.
> > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire 
> PO6 3AU
> Unless stated otherwise above:
> IBM United Kingdom Limited - Registered in England and Wales with 
> number 741598.
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20180530/50954c8f/attachment.html>


More information about the 2d-dev mailing list