[OpenJDK 2D-Dev] RFR(xxxs): 8200052: libjavajpeg: Fix compile warning in jchuff.c

[Alex Kashchenko]

Hi,

On 05/21/2018 10:14 AM, Thomas Stüfe wrote:
> On Mon, May 21, 2018 at 11:07 AM, John Paul Adrian Glaubitz
> <glaubitz at physik.fu-berlin.de> wrote:
>> On 05/18/2018 07:09 PM, Thomas Stüfe wrote:
>>> The amount of work you have to put into this far outbalances the
>>> amount of work the OpenJDK maintainers would have to spend when (if
>>> ever) they were to merge down upstream libjpeg.
>>
>> You might have more luck if you replaced libjpeg with libjpeg-turbo
>> which is used as the default libjpeg implementation in most Linux
>> distributions these days.
>>
> 
> Good idea, but not my call to make....

We ship libjpeg-turbo with RH windows builds of OpenJDK. I can say that 
it is not the nicest codebase to build (mostly asm, written for NASM; 
can be built without asm part, but that is not how most people use it). 
While linux distros give it for free, it may be not an optimal variant 
for a portable jpeg impl.

Just my 0.02 .

> 
> ..Thomas
> 
>> Adrian
>>
>> --
>>   .''`.  John Paul Adrian Glaubitz
>> : :' :  Debian Developer - glaubitz at debian.org
>> `. `'   Freie Universitaet Berlin - glaubitz at physik.fu-berlin.de
>>    `-    GPG: 62FF 8A75 84E0 2956 9546  0006 7426 3B37 F5B5 F913


-- 
-Alex