<span style=" font-size:10pt;font-family:sans-serif">Hi Phil,</span>
<br>
<br><span style=" font-size:10pt;font-family:sans-serif">As long as anyone
using png.h also includes pngpriv, it's not a problem.</span>
<br>
<br><span style=" font-size:10pt;font-family:sans-serif">If you don't include
pngpriv, the pre-process output resolves the extern lines to be extern
</span>
<br><tt><span style=" font-size:10pt">PNG_DLL_EXPORT</span></tt><span style=" font-size:10pt;font-family:sans-serif">
[function] because it appears to use the string literally if the macro
hasn't been defined.</span>
<br>
<br><span style=" font-size:10pt;font-family:sans-serif">Right now this
isn't a bug, just a quirk I was mentioning.<br>
<br>
Best Regards<br>
<br>
Adam Farley <br>
</span>
<br>
<br><tt><span style=" font-size:10pt">Phil Race <philip.race@oracle.com>
wrote on 30/05/2018 18:07:39:<br>
<br>
> From: Phil Race <philip.race@oracle.com></span></tt>
<br><tt><span style=" font-size:10pt">> To: Adam Farley8 <adam.farley@uk.ibm.com></span></tt>
<br><tt><span style=" font-size:10pt">> Cc: 2d-dev <2d-dev@openjdk.java.net></span></tt>
<br><tt><span style=" font-size:10pt">> Date: 30/05/2018 18:08</span></tt>
<br><tt><span style=" font-size:10pt">> Subject: Re: [OpenJDK 2D-Dev]
Question about splashscreen lib</span></tt>
<br><tt><span style=" font-size:10pt">> <br>
> I think PNG_DLL_EXPORT is not intended to be used directly.<br>
> I am not sure what problem this causes but it is just the way this
<br>
> library is written ...<br>
> <br>
> -phil.<br>
</span></tt>
<br><tt><span style=" font-size:10pt">> On 05/30/2018 04:16 AM, Adam
Farley8 wrote:</span></tt>
<br><tt><span style=" font-size:10pt">> Thanks Philip. <br>
> <br>
> The whole thing with PNG_DLL_EXPORT not being defined in png.h or
<br>
> png.c is still confusing to me <br>
> (pngpriv defines it), especially since png.h ends up using it post-<br>
> pre-processing, but I don't know if <br>
> that would bother people outside the minority that inspect the pre-<br>
> processed code. <br>
> <br>
> I'll leave this here as a thought for any committer bothered by <br>
> this, though I won't be pushing it. <br>
> <br>
> ---- png.h ---- <br>
> + #ifndef PNG_DLL_EXPORT <br>
> + #define PNG_DLL_EXPORT <br>
> + #endif<br>
> <br>
> Best Regards<br>
> <br>
> Adam Farley <br>
> <br>
> <br>
> Philip Race <philip.race@oracle.com> wrote on 29/05/2018 14:23:21:<br>
> <br>
> > From: Philip Race <philip.race@oracle.com> <br>
> > To: Adam Farley8 <adam.farley@uk.ibm.com> <br>
> > Cc: 2d-dev <2d-dev@openjdk.java.net> <br>
> > Date: 29/05/2018 14:23 <br>
> > Subject: Re: [OpenJDK 2D-Dev] Question about splashscreen lib
<br>
> > <br>
> > So far as I can see it does, just indirectly.<br>
> > png.c includes pngpriv.h which includes png.h<br>
> > <br>
> > Not including the declarations of the exported API in the file
that<br>
> > defines them would be a problem waiting to happen.<br>
> > <br>
> > -phil.<br>
> > <br>
> > On 5/29/18, 3:54 AM, Adam Farley8 wrote: <br>
> > Hi All, <br>
> > <br>
> > Does anyone know why png.c does not include png.h? <br>
> > <br>
> > This tripped me up recently, and I don't understand why png.c
would <br>
> > not include png.h.<br>
> > <br>
> > Best Regards<br>
> > <br>
> > Adam Farley <br>
> > <br>
> > Unless stated otherwise above:<br>
> > IBM United Kingdom Limited - Registered in England and Wales
with <br>
> > number 741598. <br>
> > Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU<br>
> Unless stated otherwise above:<br>
> IBM United Kingdom Limited - Registered in England and Wales with
<br>
> number 741598. <br>
> Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire
PO6 3AU</span></tt><span style=" font-size:10pt;font-family:sans-serif"><br>
Unless stated otherwise above:<br>
IBM United Kingdom Limited - Registered in England and Wales with number
741598. <br>
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6
3AU<br>
</span>