Re: [cp-patches] FYI: Fix getKeyPropertyListString

2007-04-08 Thread Mark Wielaard
On Sat, 2007-04-07 at 23:47 +0100, Andrew John Hughes wrote: This patch fixes the parsing in ObjectName so that it again doesn't include the wildcard '*' in the return value of getKeyPropertyListString. Should go in the release, I reckon. Thanks, installed on the release branch.

Re: 0.95 branch created

2007-04-08 Thread Roman Kennke
Hi, FAIL: javax.swing.TransferHandler.createTransferable (I thought this was fixed with Francis latest patch, but builder still has trouble with it for some reason) FAIL: javax.swing.table.JTableHeader.AccessibleJTableHeader.AccessibleJTableHeaderEntry.getFont (Not investigated yet) I'll

Re: 0.95 branch created

2007-04-08 Thread Christian Thalinger
On Sat, 2007-04-07 at 23:49 +0200, Mark Wielaard wrote: On Sat, 2007-04-07 at 00:51 +0200, Mark Wielaard wrote: FAIL: javax.swing.TransferHandler.createTransferable (I thought this was fixed with Francis latest patch, but builder still has trouble with it for some reason) This has

Method.invoke() on a non-public class from another package

2007-04-08 Thread Mark Wielaard
On Sun, 2007-04-08 at 12:53 +0200, Christian Thalinger wrote: Grrr, I hate this access checks. I'll try to fix that _again_. This seems to be pretty subtle and we found multiple runtimes (jamvm, cacao, gcj and kaffe at least) that seem to get this wrong. And the online documentation is not very

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Christian Thalinger
On Sun, 2007-04-08 at 23:12 +0200, Mark Wielaard wrote: Attached is a simplified test case (3 classes - c1 is in package p1, c2 and c3 are in package p2, c2 is not public, p1.c1 is the entry point) that should throw an IllegalAccessException on the line: m.invoke(o, new Object[0]); Sun's RI

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Robert Lougher
Hi Mark, On 4/8/07, Mark Wielaard [EMAIL PROTECTED] wrote: On Sun, 2007-04-08 at 12:53 +0200, Christian Thalinger wrote: Grrr, I hate this access checks. I'll try to fix that _again_. This seems to be pretty subtle and we found multiple runtimes (jamvm, cacao, gcj and kaffe at least) that

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Andrew Haley
Mark Wielaard writes: On Sun, 2007-04-08 at 12:53 +0200, Christian Thalinger wrote: Grrr, I hate this access checks. I'll try to fix that _again_. This seems to be pretty subtle and we found multiple runtimes (jamvm, cacao, gcj and kaffe at least) that seem to get this wrong. And the

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Christian Thalinger
On Sun, 2007-04-08 at 22:41 +0100, Robert Lougher wrote: Strangely enough, I _did_ fix this in JamVM 1.4.4, but then removed the checks again in JamVM 1.4.5! http://cvs.berlios.de/cgi-bin/viewcvs.cgi/jamvm/jamvm/src/reflect.c.diff?r1=1.9r2=1.10 If I remember the reason for removing it

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Robert Lougher
Hi, On 4/8/07, Robert Lougher [EMAIL PROTECTED] wrote: Hi Mark, On 4/8/07, Mark Wielaard [EMAIL PROTECTED] wrote: On Sun, 2007-04-08 at 12:53 +0200, Christian Thalinger wrote: Grrr, I hate this access checks. I'll try to fix that _again_. This seems to be pretty subtle and we found

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Andrew Haley
Robert Lougher writes: Anyway, I've found what it broke and my original analysis. After putting this in, BeanShell GUI stopped working with JamVM. This was found during 0.93 testing: ... I don't know where the problem lies. The class java.util.Hashtable$HashEntry is

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Christian Thalinger
On Sun, 2007-04-08 at 23:00 +0100, Robert Lougher wrote: Anyway, I've found what it broke and my original analysis. After putting this in, BeanShell GUI stopped working with JamVM. This was found during 0.93 testing: http://www.mail-archive.com/classpath@gnu.org/msg13811.html I did an

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Robert Lougher
Hi Twisti, On 4/8/07, Christian Thalinger [EMAIL PROTECTED] wrote: On Sun, 2007-04-08 at 23:00 +0100, Robert Lougher wrote: Anyway, I've found what it broke and my original analysis. After putting this in, BeanShell GUI stopped working with JamVM. This was found during 0.93 testing:

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Robert Lougher
Hi Andrew, On 4/8/07, Andrew Haley [EMAIL PROTECTED] wrote: Robert Lougher writes: Anyway, I've found what it broke and my original analysis. After putting this in, BeanShell GUI stopped working with JamVM. This was found during 0.93 testing: ... I don't know where the

Re: Method.invoke() on a non-public class from another package

2007-04-08 Thread Robert Lougher
Hi, On 4/9/07, Robert Lougher [EMAIL PROTECTED] wrote: Hi Andrew, On 4/8/07, Andrew Haley [EMAIL PROTECTED] wrote: Robert Lougher writes: Anyway, I've found what it broke and my original analysis. After putting this in, BeanShell GUI stopped working with JamVM. This was found

[commit-cp] classpath ChangeLog javax/management/ObjectName... [classpath-0_95-branch]

2007-04-08 Thread Mark Wielaard
CVSROOT:/cvsroot/classpath Module name:classpath Branch: classpath-0_95-branch Changes by: Mark Wielaard mark07/04/08 17:05:29 Modified files: . : ChangeLog javax/management: ObjectName.java Log message: 2007-04-07 Andrew John