[RFC][Icedtea6]: Update jpegclasses.patch

[Andrew Su]

----- Original Message -----
> From: "Dr Andrew John Hughes" <ahughes at redhat.com>
> To: "Andrew Su" <asu at redhat.com>
> Cc: "IcedTea" <distro-pkg-dev at openjdk.java.net>
> Sent: Monday, May 9, 2011 7:41:02 PM
> Subject: Re: [RFC][Icedtea6]: Update jpegclasses.patch
> On 18:55 Mon 09 May , Andrew Su wrote:
> >
> >
> > ----- Original Message -----
> > > From: "Andrew Su" <asu at redhat.com>
> > > To: "Dr Andrew John Hughes" <ahughes at redhat.com>
> > > Cc: "IcedTea" <distro-pkg-dev at openjdk.java.net>
> > > Sent: Wednesday, May 4, 2011 9:33:09 AM
> > > Subject: Re: [RFC][Icedtea6]: Update jpegclasses.patch
> > > ----- Original Message -----
> > > > From: "Dr Andrew John Hughes" <ahughes at redhat.com>
> > > > To: "Andrew Su" <asu at redhat.com>
> > > > Cc: "IcedTea" <distro-pkg-dev at openjdk.java.net>
> > > > Sent: Monday, May 2, 2011 6:18:24 PM
> > > > Subject: Re: [RFC][Icedtea6]: Update jpegclasses.patch
> > > > On 14:52 Mon 02 May , Andrew Su wrote:
> > > > > Hello,
> > >
> > > --snip--
> > >
> > > > I haven't looked at the code itself, but I think this would be
> > > > much
> > > > more maintainable if it used the overlays
> > > > mechanism instead, with possibly a minimal patch to enable
> > > > building
> > > > of
> > > > the code. I assume the com.sun.image.codec.jpeg
> > > > code is not present in the OpenJDK6 tarball?
> > >
> > > That's right, neither com.sun.image.codec.jpeg and
> > > sun.awt.image.codec
> > > is in the openJDK6 tarball.
> > >
> > > I agree that having it in overlays would make it easier to
> > > maintain.
> > > I've moved it there.
> > >
> > > This patch now removes the jpegclasses.patch and adds the files to
> > > overlays/jpeg.
> >
> > Hi,
> >
> > Here's an updated version with a more complete JPEGParam code. It is
> > still missing subsampling...
> > The encoder and decoder currently doesn't use all the parameters in
> > JPEGParam.
> >
> > It builds successfully and can be tested with the attached sample.
> >
> > Ok to push to icedtea6 HEAD?
> >
> > Cheers,
> >   Andrew
> 
> Yes, go ahead.

Thanks, pushed.

Question about adding Marathon to author, Matthew Flaschen is already in it, but from the bugzilla[1] we don't know of Marathon's full name.

--Andrew

[1]http://icedtea.classpath.org/bugzilla/show_bug.cgi?id=192