Merge branch 'optimize-filter-intersect' of https://github.com/kcoone…

…y/junit into kcooney-optimize-filter-intersect

src/main/java/org/junit/runner/manipulation/Filter.java

@@ -30,7 +30,12 @@ public String describe() {
 		@Override
 		public void apply(Object child) throws NoTestsRemainException {
 			// do nothing
-		};
+		}
+
+		@Override
+		public Filter intersect(Filter second) {
+			return second;
+		}
 	};
 
 	/**
@@ -89,6 +94,9 @@ public void apply(Object child) throws NoTestsRemainException {
 	 * by this Filter and {@code second}
 	 */
 	public Filter intersect(final Filter second) {
+		if (second == this || second == ALL) {
+			return this;
+		}
 		final Filter first= this;
 		return new Filter() {
 			@Override

src/test/java/org/junit/tests/manipulation/FilterTest.java

@@ -1,6 +1,7 @@
 package org.junit.tests.manipulation;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertSame;
 import org.junit.Test;
 import org.junit.runner.Description;
 import org.junit.runner.manipulation.Filter;
@@ -31,4 +32,18 @@ public void intersectionText() {
 		assertEquals("a and b", a.intersect(b).describe());
 		assertEquals("b and a", b.intersect(a).describe());
 	}
+
+	@Test
+	public void intersectSelf() {
+		NamedFilter a= new NamedFilter("a");
+		assertSame(a, a.intersect(a));
+	}
+
+	@Test
+	public void intersectAll() {
+		NamedFilter a= new NamedFilter("a");
+		assertSame(a, a.intersect(Filter.ALL));
+		assertSame(a, Filter.ALL.intersect(a));
+		assertSame(Filter.ALL, Filter.ALL.intersect(Filter.ALL));
+	}
 }