velocity-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Geir Magnusson Jr." <>
Subject Re: cvs commit: jakarta-velocity/src/java/org/apache/velocity/util/introspection
Date Mon, 10 Sep 2001 22:55:17 GMT
On 9/10/01 6:09 AM, "Attila Szegedi" <> 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 Magnusson Jr.
System and Software Consulting
Developing for the web?  See
If you look up, there are no limits - Japanese Proverb

View raw message