diff --git a/src/main/java/junit/framework/JUnit4TestAdapter.java b/src/main/java/junit/framework/JUnit4TestAdapter.java
index f5662186ff70..a05a31313c12 100644
--- a/src/main/java/junit/framework/JUnit4TestAdapter.java
+++ b/src/main/java/junit/framework/JUnit4TestAdapter.java
@@ -16,9 +16,9 @@
 public class JUnit4TestAdapter implements Test, Filterable, Sortable, Describable {
 	private final Class<?> fNewTestClass;
 
-	private Runner fRunner;
+	private final Runner fRunner;
 
-	private JUnit4TestAdapterCache fCache;
+	private final JUnit4TestAdapterCache fCache;
 
 	public JUnit4TestAdapter(Class<?> newTestClass) {
 		this(newTestClass, JUnit4TestAdapterCache.getDefault());
diff --git a/src/main/java/org/junit/rules/ExternalResource.java b/src/main/java/org/junit/rules/ExternalResource.java
index 98ac8ae0fca3..afa0c2963d2b 100644
--- a/src/main/java/org/junit/rules/ExternalResource.java
+++ b/src/main/java/org/junit/rules/ExternalResource.java
@@ -58,7 +58,6 @@ protected void before() throws Throwable {
 
 	/**
 	 * Override to tear down your specific external resource.
-	 * @throws if setup fails (which will disable {@code after}
 	 */
 	protected void after() {
 		// do nothing
diff --git a/src/test/java/junit/tests/framework/SuiteTest.java b/src/test/java/junit/tests/framework/SuiteTest.java
index 49432d7d268f..3953f4fa9786 100644
--- a/src/test/java/junit/tests/framework/SuiteTest.java
+++ b/src/test/java/junit/tests/framework/SuiteTest.java
@@ -82,7 +82,7 @@ public void testOneTestCase() {
 		assertTrue(fResult.wasSuccessful());
 	}
 	public void testOneTestCaseEclipseSeesSameStructureAs381() {
-		TestSuite t= new TestSuite(ThreeTestCases 	.class);
+		TestSuite t= new TestSuite(ThreeTestCases.class);
 		assertEquals(3, Collections.list(t.tests()).size());
 	}
 	public void testShadowedTests() {