> From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]
> By popular demand: Do not scan directories if their contents are
> excluded. Changed scannedDirs from a cache to a result and added
> a gettor method with a warning in the Javadoc that it was a testing
> method only.
Cool! Should be a boon to some builds. Thanks Matt.
> /**
> + * This method is of interest for testing purposes. The
returned
> + * Set is live and should not be modified.
> + * @return the Set of relative directory names that have been
> + * scanned.
> + */
> + public Set getScannedDirs() {
> + return scannedDirs;
> + }
How about returning Collections.unmodifiableSet(scannedDirs)
instead just to be on the safe side?
Or better yet, make getScannedDirs() package private, since
DirectoryScannerTest can still access it since in the same
package no? (albeit in a different directory.) --DD
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
