commit | e5dc67fb1de8ab0ad6af679fe7961b5ae73649ce | [log] [tgz] |
---|---|---|
author | Dave Borowitz <dborowitz@google.com> | Thu Feb 02 15:44:19 2017 -0500 |
committer | Dave Borowitz <dborowitz@google.com> | Tue Feb 07 15:45:17 2017 +0000 |
tree | e985b095b6d27e62f3cf36b8dc868c474e924c0a | |
parent | e8ac4988951806ee93bf43d9f04e2f47ec12b293 [diff] |
Assign unused Future return value to a variable Error Prone plans to make this pattern into a compile error; this avoids future breakage. Change-Id: I6a023f22f472941854a6729f6450b9b70a020069
diff --git a/src/main/java/com/googlesource/gerrit/plugins/reviewnotes/RefUpdateListener.java b/src/main/java/com/googlesource/gerrit/plugins/reviewnotes/RefUpdateListener.java index 24c9336..1a80875 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/reviewnotes/RefUpdateListener.java +++ b/src/main/java/com/googlesource/gerrit/plugins/reviewnotes/RefUpdateListener.java
@@ -86,7 +86,8 @@ } }; if (async) { - workQueue.getDefaultQueue().submit(task); + @SuppressWarnings("unused") // No assurance this completes. + Future<?> possiblyIgnoredError = workQueue.getDefaultQueue().submit(task); } else { task.run(); }