[cp-patches] RFC: Patch for duplicate entries in serialPersistentFields

2005-12-10 Thread Guilhem Lavaux
Hi, Here is a patch which (partially) fix a regression in kaffe. If you define a class like that: class A { private static final ObjectStreamField[] serialPersistentFields = { new ObjectStreamField("i", int.class), new ObjectStrea

Re: [cp-patches] RFC: Patch for duplicate entries in serialPersistentFields

2005-12-11 Thread Mark Wielaard
On Sat, 2005-12-10 at 20:16 +0100, Guilhem Lavaux wrote: > Hi, > > Here is a patch which (partially) fix a regression in kaffe. If you > define a class like that: > > class A > { > private static final ObjectStreamField[] serialPersistentFields = > { > new

Re: [cp-patches] RFC: Patch for duplicate entries in serialPersistentFields

2005-12-11 Thread Guilhem Lavaux
Mark Wielaard wrote: On Sat, 2005-12-10 at 20:16 +0100, Guilhem Lavaux wrote: Hi, Here is a patch which (partially) fix a regression in kaffe. If you define a class like that: class A { private static final ObjectStreamField[] serialPersistentFields = {

Re: [cp-patches] RFC: Patch for duplicate entries in serialPersistentFields

2005-12-11 Thread Mark Wielaard
Hi Guilhem, On Sun, 2005-12-11 at 16:41 +0100, Guilhem Lavaux wrote: > > Why not do similar and throw the InvalidClassException from > > ObjectStreamClass.lookup()? > > (But then document that clearly in our version of course.) > > Because we would not have the same API. ObjectStreamClass.lookup

Re: [cp-patches] RFC: Patch for duplicate entries in serialPersistentFields

2005-12-11 Thread Guilhem Lavaux
Hi Mark, Mark Wielaard wrote: Hi Guilhem, On Sun, 2005-12-11 at 16:41 +0100, Guilhem Lavaux wrote: Why not do similar and throw the InvalidClassException from ObjectStreamClass.lookup()? (But then document that clearly in our version of course.) Because we would not have the same API. Objec