velocity-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Geir Magnusson Jr." <ge...@optonline.net>
Subject Re: cvs commit: jakarta-velocity/src/java/org/apache/velocity/util/introspection ClassMap.java Introspector.java
Date Mon, 10 Sep 2001 22:55:17 GMT
On 9/10/01 6:09 AM, "Attila Szegedi" <szegedia@freemail.hu> wrote:

> "Map classByName" could as well be "Set cachedClassNames", storing only
> class names and using "cachedClassNames.contains(className)" instead of
> "classByName.get(name) != null". It can be somewhat confusing to code
> readers why do we store the class beside its name - we don't use the stored
> class object anywhere.

That's a good idea too...

Lets beat on this a while first, then we can nip and tuck.

geir
-- 
Geir Magnusson Jr.     geirm@optonline.net
System and Software Consulting
Developing for the web?  See http://jakarta.apache.org/velocity/
If you look up, there are no limits - Japanese Proverb


Mime
View raw message