Remove "nullness" suppressions added in CL 152864813.

They are only necessary when running the Checker Framework, and we don't.
(And they won't be necessary for the Checker Framework, either, after its' upgraded.)

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=153184952

guava-testlib/test/com/google/common/testing/FakeTickerTest.java

@@ -150,7 +150,7 @@ private void runConcurrentTest(int numberOfThreads, final Callable<Void> callabl
     final CountDownLatch startLatch = new CountDownLatch(numberOfThreads);
     final CountDownLatch doneLatch = new CountDownLatch(numberOfThreads);
     for (int i = numberOfThreads; i > 0; i--) {
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError =
           executorService.submit(
               new Callable<Void>() {

guava-tests/benchmark/com/google/common/util/concurrent/ExecutionListBenchmark.java

@@ -208,7 +208,7 @@ enum Impl {
     final AtomicInteger integer = new AtomicInteger();
     // Execute a bunch of tasks to ensure that our threads are allocated and hot
     for (int i = 0; i < NUM_THREADS * 10; i++) {
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError =
           executorService.submit(
               new Runnable() {
@@ -282,10 +282,10 @@ public Object measureSize() {
       list = impl.newExecutionList();
       listenerLatch = new CountDownLatch(numListeners * NUM_THREADS);
       for (int j = 0; j < NUM_THREADS; j++) {
-        @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+        @SuppressWarnings("unused") // go/futurereturn-lsc
         Future<?> possiblyIgnoredError = executorService.submit(addTask);
       }
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError = executorService.submit(executeTask);
       returnValue += (int) listenerLatch.getCount();
       listenerLatch.await();
@@ -305,10 +305,10 @@ public Object measureSize() {
     for (int i = 0; i < reps; i++) {
       list = impl.newExecutionList();
       listenerLatch = new CountDownLatch(numListeners * NUM_THREADS);
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError = executorService.submit(executeTask);
       for (int j = 0; j < NUM_THREADS; j++) {
-        @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+        @SuppressWarnings("unused") // go/futurereturn-lsc
         Future<?> possiblyIgnoredError1 = executorService.submit(addTask);
       }
       returnValue += (int) listenerLatch.getCount();

guava-tests/test/com/google/common/cache/CacheBuilderTest.java

@@ -477,7 +477,7 @@ public void testRemovalNotification_clear_basher() throws InterruptedException {
     final CountDownLatch tasksFinished = new CountDownLatch(nTasks);
     for (int i = 0; i < nTasks; i++) {
       final String s = "a" + i;
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError =
           threadPool.submit(
               new Runnable() {
@@ -568,7 +568,7 @@ public void testRemovalNotification_get_basher() throws InterruptedException {
 
     ExecutorService threadPool = Executors.newFixedThreadPool(nThreads);
     for (int i = 0; i < nTasks; i++) {
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError =
           threadPool.submit(
               new Runnable() {

guava-tests/test/com/google/common/cache/CacheLoaderTest.java

@@ -78,7 +78,7 @@ public Map<Object, Object> loadAll(Iterable<? extends Object> keys) {
     assertEquals(0, loadAllCount.get());
 
     baseLoader.load(new Object());
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError = baseLoader.reload(new Object(), new Object());
     baseLoader.loadAll(ImmutableList.of(new Object()));
     assertEquals(1, loadCount.get());
@@ -90,7 +90,7 @@ public Map<Object, Object> loadAll(Iterable<? extends Object> keys) {
         CacheLoader.asyncReloading(baseLoader, executor);
 
     asyncReloader.load(new Object());
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError1 = asyncReloader.reload(new Object(), new Object());
     asyncReloader.loadAll(ImmutableList.of(new Object()));
     assertEquals(2, loadCount.get());

guava-tests/test/com/google/common/collect/QueuesTest.java

@@ -101,15 +101,15 @@ public void testMultipleProducers() throws Exception {
 
   private void testMultipleProducers(BlockingQueue<Object> q) throws InterruptedException {
     for (boolean interruptibly : new boolean[] {true, false}) {
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError = threadPool.submit(new Producer(q, 20));
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError1 = threadPool.submit(new Producer(q, 20));
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError2 = threadPool.submit(new Producer(q, 20));
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError3 = threadPool.submit(new Producer(q, 20));
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError4 = threadPool.submit(new Producer(q, 20));
 
       List<Object> buf = newArrayList();
@@ -182,7 +182,7 @@ public void testNegativeMaxElements() throws Exception {
   }
 
   private void testNegativeMaxElements(BlockingQueue<Object> q) throws InterruptedException {
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError = threadPool.submit(new Producer(q, 1));
 
     List<Object> buf = newArrayList();
@@ -201,7 +201,7 @@ public void testDrain_throws() throws Exception {
   }
 
   private void testDrain_throws(BlockingQueue<Object> q) {
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError = threadPool.submit(new Interrupter(currentThread()));
     try {
       Queues.drain(q, ImmutableList.of(), 100, MAX_VALUE, NANOSECONDS);
@@ -218,7 +218,7 @@ public void testDrainUninterruptibly_doesNotThrow() throws Exception {
 
   private void testDrainUninterruptibly_doesNotThrow(final BlockingQueue<Object> q) {
     final Thread mainThread = currentThread();
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError =
         threadPool.submit(
             new Callable<Void>() {
@@ -275,7 +275,7 @@ private void assertInterruptibleDrained(BlockingQueue<Object> q) {
     }
 
     // but does the wait actually occurs?
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError = threadPool.submit(new Interrupter(currentThread()));
     try {
       // if waiting works, this should get stuck
@@ -291,7 +291,7 @@ private void assertUninterruptibleDrained(BlockingQueue<Object> q) {
     assertEquals(0, Queues.drainUninterruptibly(q, ImmutableList.of(), 0, 10, MILLISECONDS));
 
     // but does the wait actually occurs?
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError = threadPool.submit(new Interrupter(currentThread()));
 
     Stopwatch timer = Stopwatch.createStarted();

guava-tests/test/com/google/common/io/MoreFilesTest.java

@@ -627,7 +627,7 @@ public void testDeleteRecursively_nonDirectoryFile() throws IOException {
    */
   private static void startDirectorySymlinkSwitching(
       final Path file, final Path target, ExecutorService executor) {
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError =
         executor.submit(
             new Runnable() {

guava-tests/test/com/google/common/util/concurrent/AbstractFutureTest.java

@@ -379,7 +379,7 @@ public void testFutureBash() {
       final AbstractFuture<String> future = new AbstractFuture<String>() {};
       currentFuture.set(future);
       for (Callable<?> task : allTasks) {
-        @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+        @SuppressWarnings("unused") // go/futurereturn-lsc
         Future<?> possiblyIgnoredError = executor.submit(task);
       }
       awaitUnchecked(barrier);
@@ -580,7 +580,7 @@ public void testSetFutureCancelBash_withDoneFuture() {
       final AbstractFuture<String> future = new AbstractFuture<String>() {};
       currentFuture.set(future);
       for (Callable<?> task : allTasks) {
-        @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+        @SuppressWarnings("unused") // go/futurereturn-lsc
         Future<?> possiblyIgnoredError = executor.submit(task);
       }
       awaitUnchecked(barrier);

guava-tests/test/com/google/common/util/concurrent/AtomicLongMapTest.java

@@ -592,7 +592,7 @@ public void testModify_basher() throws InterruptedException {
 
     ExecutorService threadPool = Executors.newFixedThreadPool(nThreads);
     for (int i = 0; i < nTasks; i++) {
-      @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+      @SuppressWarnings("unused") // go/futurereturn-lsc
       Future<?> possiblyIgnoredError =
           threadPool.submit(
               new Runnable() {

guava-tests/test/com/google/common/util/concurrent/FuturesTest.java

@@ -1725,7 +1725,7 @@ public ListenableFuture<Integer> apply(String input) throws Exception {
 
     // Pause the executor.
     final CountDownLatch beforeFunction = new CountDownLatch(1);
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError =
         executor.submit(
             new Runnable() {

guava-tests/test/com/google/common/util/concurrent/WrappingScheduledExecutorServiceTest.java

@@ -46,11 +46,11 @@ public void testSchedule() {
     MockExecutor mock = new MockExecutor();
     TestExecutor testExecutor = new TestExecutor(mock);
 
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError = testExecutor.schedule(DO_NOTHING, 10, TimeUnit.MINUTES);
     mock.assertLastMethodCalled("scheduleRunnable", 10, TimeUnit.MINUTES);
 
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError1 =
         testExecutor.schedule(Executors.callable(DO_NOTHING), 5, TimeUnit.SECONDS);
     mock.assertLastMethodCalled("scheduleCallable", 5, TimeUnit.SECONDS);
@@ -59,12 +59,12 @@ public void testSchedule() {
   public void testSchedule_repeating() {
     MockExecutor mock = new MockExecutor();
     TestExecutor testExecutor = new TestExecutor(mock);
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError =
         testExecutor.scheduleWithFixedDelay(DO_NOTHING, 100, 10, TimeUnit.MINUTES);
     mock.assertLastMethodCalled("scheduleWithFixedDelay", 100, 10, TimeUnit.MINUTES);
 
-    @SuppressWarnings({"unused", "nullness"}) // go/futurereturn-lsc
+    @SuppressWarnings("unused") // go/futurereturn-lsc
     Future<?> possiblyIgnoredError1 =
         testExecutor.scheduleAtFixedRate(DO_NOTHING, 3, 7, TimeUnit.SECONDS);
     mock.assertLastMethodCalled("scheduleAtFixedRate", 3, 7, TimeUnit.SECONDS);