DrJava Legacy SVN Repository

This revision cleans up the _testFileSort method in
DefaultCompilerModel.java and marks the _size field in Gap.java as
volatile. The following files were modified:

M src/edu/rice/cs/drjava/model/definitions/reducedmodel/Gap.java
M src/edu/rice/cs/drjava/model/compiler/DefaultCompilerModel.java

Commented out log that was breaking the build.

This revision includes a partial workaround for a bug in the handling
of forward references in javalanglevels.jar.

The following files were modified:

M src/edu/rice/cs/drjava/model/compiler/DefaultCompilerModel.java

Cleaned up the formatting of LanguageLevelConverter.java and
LanguageLevelVisitor.java---eliminating some egregiously long lines.

The following files were modified:

M src/edu/rice/cs/javalanglevels/LanguageLevelConverter.java
M src/edu/rice/cs/javalanglevels/LanguageLevelVisitor.java

Updated javalanglevels-base.jar after Corky's commit.

Added defensive null tests for line 963 in ExpressionTypeChecker.

M src/edu/rice/cs/javalanglevels/LValueTypeChecker.java
M src/edu/rice/cs/javalanglevels/ExpressionTypeChecker.java
M src/edu/rice/cs/javalanglevels/LanguageLevelVisitor.java

Updated library after Corky's commit.

M drjava/lib/javalanglevels-base.jar

Added a defensive null check in LanguageLevelConverter and LanguageLevelVisitor to prevent NullPointerException.

Fixed bug 2634638: Next/Previous Word has Problems with Whitespace

The main problem was that JTextComponent.getText() translates newlines to System.getProperty("line.separator"), which is two characters long on Windows. The internal offset, however, is just based on "\n" newlines. Therefore, the offset always got one character too large per line. Instead, we are now using JTextComponent.getDocument().getText(...), which keeps the internal representation.

Also fixed two problems with boundary conditions.

Fixed a bug in a Suspension embedded inside the constructor for
MovingDocumentRegion; the code failed to consider the possibility tat
a match might occur at an offset in a line (excerpt) beyond our
maximum offset length (120). Only the MovingDocumentRegions source
file was modifed.