[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [cp-patches] StringBuilder and Appendable for CVS HEAD
From: |
Andrew John Hughes |
Subject: |
Re: [cp-patches] StringBuilder and Appendable for CVS HEAD |
Date: |
Mon, 13 Dec 2004 16:30:55 +0000 |
On Mon, 2004-12-13 at 16:07, Michael Koch wrote:
> Am Montag, 13. Dezember 2004 16:53 schrieb Andrew John Hughes:
> > Attached is a patch to add StringBuilder (a non-threaded derivative
> > of StringBuffer) and Appendable, an additional interface it
> > implements. Currently, there is a small workaround so that the
> > implementation of Appendable will compile. The return type of
> > these methods have had to be typed to Appendable rather than
> > StringBuilder at present, as only 1.5 implements co-variant return
> > types. This can easily be fixed when Classpath compilers implement
> > this, and using code only needs an explicit downcast to
> > StringBuilder to work with this change.
> >
> > Anyway, let me know your thoughts ;)
>
> There should at least be FIXME comments where we need to change the
> return types later.
>
>
> Michael
My thoughts exactly, which is why I put them in ;)
It doesn't affect using it as a drop-in StringBuffer replacement, only
in using it via the Appendable interface (which is only in 1.5 and this
patch itself).
--
Andrew :-)
Please avoid sending me Microsoft Office (e.g. Word, PowerPoint) attachments.
See http://www.fsf.org/philosophy/no-word-attachments.html
No software patents in Europe -- http://nosoftwarepatents.com
"Value your freedom, or you will lose it, teaches history.
`Don't bother us with politics' respond those who don't want to learn."
-- Richard Stallman
"We've all been part of the biggest beta test the world has ever known --
Windows"
-- Victor Wheatman, Gartner
signature.asc
Description: This is a digitally signed message part
RE: [cp-patches] StringBuilder and Appendable for CVS HEAD, Jeroen Frijters, 2004/12/15