Browse Source

Various terseness like I tend to do;

More importantly, fixed Gump failure on Unix (see end of change list).
Dyslexia strikes!


git-svn-id: https://svn.apache.org/repos/asf/ant/core/trunk@277789 13f79535-47bb-0310-9956-ffa450edef68
master
Matthew Jason Benson 20 years ago
parent
commit
7721d8606c
1 changed files with 9 additions and 25 deletions
  1. +9
    -25
      src/main/org/apache/tools/ant/filters/FixCrLfFilter.java

+ 9
- 25
src/main/org/apache/tools/ant/filters/FixCrLfFilter.java View File

@@ -406,18 +406,12 @@ public final class FixCrLfFilter
* Does this filter want to block edits on the last character returned by read()? * Does this filter want to block edits on the last character returned by read()?
*/ */
public boolean editsBlocked() { public boolean editsBlocked() {
if (in instanceof SimpleFilterReader) {
return ((SimpleFilterReader) in).editsBlocked();
}
return false;
return in instanceof SimpleFilterReader
&& ((SimpleFilterReader) in).editsBlocked();
} }


public int read() throws java.io.IOException { public int read() throws java.io.IOException {
if (preemptIndex > 0) {
return preempt[--preemptIndex];
}

return in.read();
return preemptIndex > 0 ? preempt[--preemptIndex] : in.read();
} }


public void close() throws java.io.IOException { public void close() throws java.io.IOException {
@@ -457,10 +451,7 @@ public final class FixCrLfFilter
count++; count++;
} }
// if at EOF with no characters in the buffer, return EOF // if at EOF with no characters in the buffer, return EOF
if (count == 0 && c == -1) {
return -1;
}
return count;
return (count == 0 && c == -1) ? -1 : count;
} }
} }


@@ -610,7 +601,6 @@ public final class FixCrLfFilter
} else { } else {
normalizedEOL--; normalizedEOL--;
} }

return thisChar; return thisChar;
} }
} }
@@ -637,7 +627,6 @@ public final class FixCrLfFilter
push(eol, 1, eol.length - 1); push(eol, 1, eol.length - 1);
} }
} }
lastChar = thisChar; lastChar = thisChar;
return thisChar; return thisChar;
} }
@@ -654,14 +643,10 @@ public final class FixCrLfFilter
int thisChar = super.read(); int thisChar = super.read();


// if source is EOF but last character was NOT ctrl-z, return ctrl-z // if source is EOF but last character was NOT ctrl-z, return ctrl-z
if (thisChar == -1) {
if (lastChar != CTRLZ) {
lastChar = CTRLZ;
thisChar = CTRLZ;
}
} else {
lastChar = thisChar;
if (thisChar == -1 && lastChar != CTRLZ) {
thisChar = CTRLZ;
} }
lastChar = thisChar;
return thisChar; return thisChar;
} }
} }
@@ -790,7 +775,6 @@ public final class FixCrLfFilter
for (;width > 1; width--) { for (;width > 1; width--) {
push(' '); push(' ');
} }

c = ' '; c = ' ';
} }
columnNumber += width; columnNumber += width;
@@ -873,13 +857,13 @@ public final class FixCrLfFilter
if (this.equals(ASIS)) { if (this.equals(ASIS)) {
return ASIS; return ASIS;
} }
if (this.equals(CR) || this.equals(UNIX)) {
if (this.equals(CR) || this.equals(MAC)) {
return CR; return CR;
} }
if (this.equals(CRLF) || this.equals(DOS)) { if (this.equals(CRLF) || this.equals(DOS)) {
return CRLF; return CRLF;
} }
if (this.equals(LF) || this.equals(MAC)) {
if (this.equals(LF) || this.equals(UNIX)) {
return LF; return LF;
} }
throw new IllegalStateException("No replacement for " + this); throw new IllegalStateException("No replacement for " + this);


Loading…
Cancel
Save