[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Thu, 07 Aug 2003 15:56:46 +0200
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3.1) Gecko/20030428
Mark Wielaard wrote:
Once I have the agreement I will submit the necessary modifications to
make the serialization a little
stronger (well at least it will make kaffe's regression test pass).
There are still one or two issues like the
fact the serializer does not check invalid serialPersistentFields but it
nearly behaves rightfully.
Besides you suppress the check for
of the field although it is quite sure it will not be there (I did not
find it in the all source
code of classpath): this causes some huge NullPointerException problems.
The only solution is to change the name "getSerialPersistentFields" into
but I do not see the point to complicate things... :( (if someone can
explain it to me)
The only user of the method is setFields() which does the check before
calling getSerialPersistentFields() (it checks by also doing a
getDeclaredField, but this time with the correct spelling).
I agree that the code is not that clean.