Reviewer needed: backport of "6296893: BMP Writer handles TopDown property incorrectly for some of the compression types" to IcedTea6 HEAD

[Dr Andrew John Hughes]

On 17:36 Thu 08 Sep     , Pavel Tisnovsky wrote:
> Dr Andrew John Hughes wrote:
> > On 17:12 Thu 08 Sep     , Pavel Tisnovsky wrote:
> >> Greetings,
> >>
> >> here's another backport I'd like to push to IcedTea6 HEAD:
> >>
> >> 6296893: BMP Writer handles TopDown property incorrectly for some of the
> >> compression types
> >>
> >> It fixes the BMP writer behaviour. The regression test contained in this
> >> backport works correctly. Tested on RHEL 5 x86_64.
> >>
> >>
> >> ChangeLog entry:
> >>
> >> 2011-09-08  Pavel Tisnovsky  <ptisnovs at redhat.com>
> >>
> >>         * Makefile.am: added new patch
> >>         * NEWS: updated with backport
> >>         * patches/openjdk/6296893-BMP_Writer_handles_TopDown_prop_incorrectly.patch:
> >>         Backport of 6296893: BMP Writer handles TopDown property incorrectly
> >>         for some of the compression types
> >>
> >>
> >>
> >> Can anybody please review this patch?
> >>
> >> Thank you in advance,
> >> Pavel
> > 
> > This looks fine.
> thank you
> > 
> > What's your progress with getting some of these into OpenJDK6?
> 
> I thought that OpenJDK6 is now closed for new patches, but I could try to sent
> some fixes to the appropriate mail lists.

No, only OpenJDK7 was ever closed during the period before the release, but that's
now open again via the 7u trees.
-- 
Andrew :)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and IcedTea
http://www.gnu.org/software/classpath
http://icedtea.classpath.org
PGP Key: F5862A37 (https://keys.indymedia.org/)
Fingerprint = EA30 D855 D50F 90CD F54D  0698 0713 C3ED F586 2A37