[OpenJDK 2D-Dev] Dont use libjpeg-internal macros in splashscreen

[Phil Race]

Looks OK to me, but adding awt-dev as splashscreen is owned by AWT.

-phil.

On 4/30/2014 1:48 PM, Omair Majid wrote:
> Hi,
>
> As indicated in the libjpeg API documentation [1], the standard libjpeg
> headers are: jpeglib.h, jerror.h, jconfig.h, jmorecfg.h.
>
> splashscreen_jpeg.c violates this layering and uses the SIZEOF macro
> from the non-public header jinclude.h directly. This macro is a wrapper
> over sizeof. The code in splashscreen_jpeg.c already uses sizeof in
> multiple places so it seems like it should be okay to use it in some
> more.
>
> The following webrev fixes it:
> http://cr.openjdk.java.net/~omajid/webrevs/libjpeg-internal-macros/01/
>
> One motivation of having this patch is that if/when we add
> system-libjpeg support, the header jinclude.h will become inaccessible.
>
> Thanks,
> Omair
>
> [1] http://svn.code.sf.net/p/libjpeg-turbo/code/trunk/libjpeg.txt
>