Changeset 11465 in josm for trunk

Timestamp:

2017-01-15T23:55:54+01:00 (8 years ago)

Author:

wiktorn

Message:

Wait in loop - properly handle thread interruptions

File:

• trunk/test/unit/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJobTest.java (modified) (3 diffs)

trunk/test/unit/org/openstreetmap/josm/data/cache/JCSCachedTileLoaderJobTest.java

@@ -100 +100 @@
      */
     @Test
-    @SuppressFBWarnings(value = "WA_NOT_IN_LOOP")
-    public void testUnknownHost() throws IOException, InterruptedException {
+    public void testUnknownHost() throws IOException {
         String key = "key_unknown_host";
         TestCachedTileLoaderJob job = new TestCachedTileLoaderJob("http://unkownhost.unkownhost/unkown", key);
@@ -107 +106 @@
         job.submit(listener, true);
         synchronized (listener) {
-            if (!listener.ready) {
-                listener.wait();
+            while (!listener.ready) {
+                try {
+                    listener.wait();
+                } catch (InterruptedException e1) {
+                    // do nothing, still wait
+                }
             }
         }
@@ -124 +127 @@
         job.submit(listener, true);
         synchronized (listener) {
-            if (!listener.ready) {
-                listener.wait();
+            while (!listener.ready) {
+                try {
+                    listener.wait();
+                } catch (InterruptedException e1) {
+                    // do nothing, wait
+                }
             }
         }