On Wed, 23 Aug 2006, Kev Jackson <[EMAIL PROTECTED]> wrote:

> I want to run this by people here to see what people think of this.
> Basically DirectoryScanner has some duplicated code [
> accountForIncludedFile, accountForIncludedDir ], I think I have a
> refactored processIncluded method that can replace the duplication,
> but it relies on using Reflection to get access to the classes
> fields and then does the work [1].

Please run some tests on really large directory trees,
DirectoryScanner is probably the class with the biggest impact on
perceived Ant performance.  We shouldn't do anything that makes it
significantly slower.

Stefan

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to