On 09:42 Thu 09 Dec , Andrew Haley wrote: > On 12/08/2010 05:17 PM, Dr Andrew John Hughes wrote: > > On 11:13 Wed 08 Dec , Andrew Haley wrote: > >> On 12/08/2010 10:58 AM, Pekka Enberg wrote: > >>> On Wed, Dec 8, 2010 at 12:32 PM, Andrew Haley <aph@xxxxxxxxxx> wrote: > >>>> I hereby offer to review some patches. Please send pointers to the > >>>> list. > >>> > >>> http://developer.classpath.org/pipermail/classpath-patches/2010-November/006511.html > >> > >> This needs a ChangeLog, otherwise OK. > >> > >>> http://developer.classpath.org/pipermail/classpath-patches/2010-November/006513.html > >> > >> This needs a ChangeLog, otherwise OK. > >> > > > > I've already seen these and I agree both seem fine. But Pekka needs a copyright > > assignment before any more work can be committed. > > > >>> http://developer.classpath.org/pipermail/classpath-patches/2010-November/006512.html > >> > >> What compatibility problem does this fix? > >> > > > > I'd like to see a test case in Mauve for this before we change this. > > That's not a good reason not to commit: the fix is in other versions, > including gcj. > I disagree. Just because the version in gcj is different doesn't mean it's correct. As far as I'm aware, Pekka already has a testcase for this so it would be good to have it in Mauve. > Andrew. -- 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: 94EFD9D8 (http://subkeys.pgp.net) Fingerprint = F8EF F1EA 401E 2E60 15FA 7927 142C 2591 94EF D9D8