Menu

Commit [r4872]  Maximize  Restore  History

Fixes bug 2738438: Saving File with class, but no Identifier throws Exception

Improves detection of empty files that trip up unit testing: If the
document doesn't produce a *.class file, then this will be considered
"out of sync" even though it isn't. We don't really have many options
here unless we actually parse the document.

As a heuristic, we check if there is any non-comment text that
includes the words "class", "interface" or "enum". If the file
doesn't, we presume it is empty and doesn't generate a class file;
therefore, it cannot ever be out of sync.

Improved error diagnostics if classes are out of sync before JUnit: The documents
in question are now actually displayed, and the error messages are more
precise in case compiling fails.

Moved MainFrame.setPopupLoc to Utilities.setPopupLoc. It doesn't
have anything to do with MainFrame.

mgricken 2009-04-06

1 2 > >> (Page 1 of 2)
changed /trunk/drjava/src/edu/rice/cs/drjava/model/AbstractGlobalModel.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/DummyGlobalModel.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/DummyGlobalModelListener.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/DummyOpenDefDoc.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/GlobalEventNotifier.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/GlobalModel.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/GlobalModelJUnitTest.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/GlobalModelTestCase.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/OpenDefinitionsDocument.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/definitions/DefinitionsDocument.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/definitions/DefinitionsDocumentTest.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/junit/DefaultJUnitModel.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitEventNotifier.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitListener.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitModel.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitModelCallback.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/InterpreterJVM.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/MainJVM.java
changed /trunk/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/MainJVMRemoteI.java
changed /trunk/drjava/src/edu/rice/cs/drjava/ui/AboutDialog.java
changed /trunk/drjava/src/edu/rice/cs/drjava/ui/ClipboardHistoryFrame.java
changed /trunk/drjava/src/edu/rice/cs/drjava/ui/DetachedFrame.java
changed /trunk/drjava/src/edu/rice/cs/drjava/ui/DrJavaErrorHandler.java
changed /trunk/drjava/src/edu/rice/cs/drjava/ui/DrJavaErrorPopup.java
changed /trunk/drjava/src/edu/rice/cs/drjava/ui/EditExternalDialog.java
/trunk/drjava/src/edu/rice/cs/drjava/model/AbstractGlobalModel.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/DummyGlobalModel.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/DummyGlobalModelListener.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/DummyOpenDefDoc.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/GlobalEventNotifier.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/GlobalModel.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/GlobalModelJUnitTest.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/GlobalModelTestCase.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/OpenDefinitionsDocument.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/definitions/DefinitionsDocument.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/definitions/DefinitionsDocumentTest.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/junit/DefaultJUnitModel.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitEventNotifier.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitListener.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitModel.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/junit/JUnitModelCallback.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/InterpreterJVM.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/MainJVM.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/model/repl/newjvm/MainJVMRemoteI.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/ui/AboutDialog.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/ui/ClipboardHistoryFrame.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/ui/DetachedFrame.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/ui/DrJavaErrorHandler.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/ui/DrJavaErrorPopup.java Diff Switch to side-by-side view
Loading...
/trunk/drjava/src/edu/rice/cs/drjava/ui/EditExternalDialog.java Diff Switch to side-by-side view
Loading...
1 2 > >> (Page 1 of 2)
MongoDB Logo MongoDB