From: Franklin, Brian <brian.franklin_at_ciraden.com>
Date: Thu, 10 Jul 2003 11:01:07 -0400

Kohsuke, I guess this is mainly directed to you.

Thanks for the synchronized methods extension! So far, it seems to be
working great. I'll be able to let you know more in the near future.

Of course, while satisfied, the customer always wants more. 8-} So I have
another suggestion for how to enhance it. When the synchronized methods
extension is active, could all elements that were Lists be initialized with
"... = Collections.synchronizedList( new ListImpl(new ArrayList()) );"
instead of just the ListImpl?

As it is, any time the contents of one of those List properties is accessed
(for an add/remove/etc), you have to be sure to synchronize on it. The
Collections.synchronizedList wrapping would at least narrow that down so
that you only have to synchronize on the List when iterating.

Let me know what you think, and thanks again for the feature.

Brian Franklin