You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@xalan.apache.org by cu...@apache.org on 2002/03/11 14:29:57 UTC
cvs commit: xml-xalan/test/java/src/org/apache/qetest DirFilter.java
curcuru 02/03/11 05:29:57
Modified: test/java/src/org/apache/qetest DirFilter.java
Log:
Fixup logic for setUseIncludesOnly(); now only turns it true if
a user explicitly sets the includes; refactor: should be moved to
parent IncludeExcludeFilter class later
Revision Changes Path
1.2 +9 -3 xml-xalan/test/java/src/org/apache/qetest/DirFilter.java
Index: DirFilter.java
===================================================================
RCS file: /home/cvs/xml-xalan/test/java/src/org/apache/qetest/DirFilter.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- DirFilter.java 8 Mar 2002 15:13:04 -0000 1.1
+++ DirFilter.java 11 Mar 2002 13:29:56 -0000 1.2
@@ -65,7 +65,7 @@
* <p>Returns directories that match our includes/excludes.</p>
*
* @author shane_curcuru@us.ibm.com
- * @version $Id: DirFilter.java,v 1.1 2002/03/08 15:13:04 curcuru Exp $
+ * @version $Id: DirFilter.java,v 1.2 2002/03/11 13:29:56 curcuru Exp $
*/
public class DirFilter extends IncludeExcludeFilter
{
@@ -73,7 +73,8 @@
/** Initialize for defaults (not using inclusion list). */
public DirFilter()
{
- setUseIncludesOnly(true);
+ // Default is to not use includes; just get all dirs
+ setUseIncludesOnly(false);
}
/**
@@ -86,7 +87,12 @@
{
setIncludes(inc);
setExcludes(exc);
- setUseIncludesOnly(true);
+ // Only use includes only if they're set
+ //refactor: logic might better be put in IncludeExcludeFilter
+ if ((null != inc) && (inc.length() > 0))
+ setUseIncludesOnly(true);
+ else
+ setUseIncludesOnly(false);
}
/**
---------------------------------------------------------------------
To unsubscribe, e-mail: xalan-cvs-unsubscribe@xml.apache.org
For additional commands, e-mail: xalan-cvs-help@xml.apache.org