Re: [PATCH] Serialization #5

[Brian Jones]

Guilhem Lavaux <address@hidden> writes:

> Dalibor Topic wrote:
> 
> > Salut Guilhem,
> >
> >> @@ -442,18 +443,46 @@
> >>      String field_name = this.realInputStream.readUTF ();
> >>      dumpElementln (field_name);
> >>      String class_name;
> >> -          +   +    // There're many cases you can't get
> >> java.lang.Class from
> >> +    // typename if your context class loader can't load it,
> >> +    // then use typename to construct the field
> >> +    // GL => No. You lose the capability to access the class loader.
> >> +    // Type resolution must be done here. If it is an object we
> >> +    // create the class just here if it is something else we delegate
> >> +    // to TypeSignature.
> >
> >
> > Just rewrite the comment, don't discuss it in another. ;)
> 
> 
> Ok. :-) Fixed in the next commit.
> 
> Cheers,
> Guilhem.

Guilhem are you going to be or have committed these to Classpath CVS?

Thanks,
Brian
-- 
Brian Jones <address@hidden>