aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java
diff options
context:
space:
mode:
authorGravatar brandjon <brandjon@google.com>2017-07-11 22:06:22 +0200
committerGravatar László Csomor <laszlocsomor@google.com>2017-07-12 08:50:03 +0200
commited249612955ea24b459425204642022da9936fb3 (patch)
tree6c7e9f048c742113514497f803c33d52905667c3 /src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java
parent3b038b9f7657f2d4c77700964ca06d92b3a6db12 (diff)
Remove Java 7 type inference workarounds and TODOs
Found by searching for TODOs mentioning "Java 7". There are plenty of other instances where we can simplify, but I don't want to be responsible for a large blast radius. At least in these cases we're fixing outdated comments. RELNOTES: None PiperOrigin-RevId: 161570189
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java9
1 files changed, 2 insertions, 7 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java b/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java
index 72829ee07b..55d607bc06 100644
--- a/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java
+++ b/src/main/java/com/google/devtools/build/lib/buildtool/buildevent/TestFilteringCompleteEvent.java
@@ -42,13 +42,8 @@ public class TestFilteringCompleteEvent {
public TestFilteringCompleteEvent(
Collection<? extends ConfiguredTarget> targets,
Collection<? extends ConfiguredTarget> testTargets) {
- // Do not remove <ConfiguredTarget>: workaround for Java 7 type inference.
- this.targets = ImmutableList.<ConfiguredTarget>copyOf(targets);
- this.testTargets =
- testTargets == null
- ? null
- // Do not remove <ConfiguredTarget>: workaround for Java 7 type inference.
- : ImmutableList.<ConfiguredTarget>copyOf(testTargets);
+ this.targets = ImmutableList.copyOf(targets);
+ this.testTargets = testTargets == null ? null : ImmutableList.copyOf(testTargets);
if (testTargets == null) {
return;
}