Adam,
Thanks for the last point I was not aware of that; thanks for examples of use
also.
Jacques
From: "Adam Heath" <[EMAIL PROTECTED]>
[EMAIL PROTECTED] wrote:
Author: lektran
Date: Fri Aug 15 14:40:30 2008
New Revision: 686377
URL: http://svn.apache.org/viewvc?rev=686377&view=rev
Log:
Various clean ups, no functional changes
While not having any specific feedback on this revision, I thought I
might as well mention what code changes I have been working on.
Adding generics markup(this is the biggest one).
Switching from StringBuffer to StringBuilder
Enhanced for(Iterable stuff)
Getting rid of new (Boolean|Byte|Double|Float|Integer|Long|Short)
Switching to FastList|FastMap|FastSet where appropriate
Fixing string +, to use StringBuilder, in loops
I do this, not by looking at any compiler warnings, but by editting
every single file, and looking at every line.
One thing I'd like to mention about generics; you may have noticed that
in some places, there is a Map<String, Object>, and in others, a
Map<String, ? extends Object>, and wondered what the different is. In
the latter case, the simple explanation is that it effectively stops the
code from inserting *new* items into the map. Ie, it makes it
read-only. You can still remove and fetch items, however. (if you are
interested in a more detailed explanation, just ask).
I use this to tell generic services that they should not modify the
incoming context. It is also used when fetching a list of elements from
a DOM.