Re: [PATCH] Serialization #5

[Guilhem Lavaux]

Brian Jones wrote:

> 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?
>  

If CVS is back online I'm going to commit these changes.

BTW, I'd like also to commit RuleBasedCollator but I'll send an e-mail 
later about it.

Cheers,
Guilhem.