mbenson     2004/12/10 13:49:32

  Modified:    src/main/org/apache/tools/ant/types AbstractFileSet.java
  Log:
  More LOC bumming
  
  Revision  Changes    Path
  1.36      +14 -27    
ant/src/main/org/apache/tools/ant/types/AbstractFileSet.java
  
  Index: AbstractFileSet.java
  ===================================================================
  RCS file: 
/home/cvs/ant/src/main/org/apache/tools/ant/types/AbstractFileSet.java,v
  retrieving revision 1.35
  retrieving revision 1.36
  diff -u -r1.35 -r1.36
  --- AbstractFileSet.java      10 Dec 2004 18:28:45 -0000      1.35
  +++ AbstractFileSet.java      10 Dec 2004 21:49:32 -0000      1.36
  @@ -54,8 +54,8 @@
    * <p>Common base class for DirSet and FileSet.</p>
    *
    */
  -public abstract class AbstractFileSet extends DataType implements Cloneable,
  -        SelectorContainer {
  +public abstract class AbstractFileSet extends DataType
  +    implements Cloneable, SelectorContainer {
   
       private PatternSet defaultPatterns = new PatternSet();
       private Vector additionalPatterns = new Vector();
  @@ -127,10 +127,7 @@
        * @return <code>File</code>.
        */
       public File getDir(Project p) {
  -        if (isReference()) {
  -            return getRef(p).getDir(p);
  -        }
  -        return dir;
  +        return (isReference()) ? getRef(p).getDir(p) : dir;
       }
   
       /**
  @@ -400,12 +397,10 @@
           }
           Object o = getRefid().getReferencedObject(p);
           if (!getClass().isAssignableFrom(o.getClass())) {
  -            String msg = getRefid().getRefId() + " doesn\'t denote a "
  -                + getDataTypeName();
  -            throw new BuildException(msg);
  -        } else {
  -            return (AbstractFileSet) o;
  +            throw new BuildException(getRefid().getRefId()
  +                + " doesn\'t denote a " + getDataTypeName());
           }
  +        return (AbstractFileSet) o;
       }
   
       // SelectorContainer methods
  @@ -416,10 +411,8 @@
        * @return whether any selectors are in this container.
        */
       public boolean hasSelectors() {
  -        if (isReference() && getProject() != null) {
  -            return getRef(getProject()).hasSelectors();
  -        }
  -        return !(selectors.isEmpty());
  +        return (isReference() && getProject() != null)
  +            ? getRef(getProject()).hasSelectors() : !(selectors.isEmpty());
       }
   
       /**
  @@ -450,10 +443,8 @@
        * @return the number of selectors in this container as an 
<code>int</code>.
        */
       public int selectorCount() {
  -        if (isReference() && getProject() != null) {
  -            return getRef(getProject()).selectorCount();
  -        }
  -        return selectors.size();
  +        return (isReference() && getProject() != null)
  +            ? getRef(getProject()).selectorCount() : selectors.size();
       }
   
       /**
  @@ -462,10 +453,8 @@
        * @return a <code>FileSelector[]</code> of the selectors in this 
container.
        */
       public FileSelector[] getSelectors(Project p) {
  -        if (isReference()) {
  -            return getRef(p).getSelectors(p);
  -        }
  -        return (FileSelector[])(selectors.toArray(
  +        return (isReference())
  +            ? getRef(p).getSelectors(p) : (FileSelector[])(selectors.toArray(
               new FileSelector[selectors.size()]));
       }
   
  @@ -475,10 +464,8 @@
        * @return an <code>Enumeration</code> of selectors.
        */
       public Enumeration selectorElements() {
  -        if (isReference() && getProject() != null) {
  -            return getRef(getProject()).selectorElements();
  -        }
  -        return selectors.elements();
  +        return (isReference() && getProject() != null)
  +            ? getRef(getProject()).selectorElements() : selectors.elements();
       }
   
       /**
  
  
  

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

Reply via email to