Re: RFR: 8023326 [TESTBUG] java/net/CookieHandler/LocalHostCookie.java misplaced try/finally

[Chris Hegarty]

Looks ok to me Mark.

-Chris.

On 08/21/2013 02:37 PM, Mark Sheppard wrote:

Hi
please oblige and review the fix below to address the issue in JDK-8023326
which makes the test more robust to certain exceptions thrown.

http://cr.openjdk.java.net/~msheppar/8023326/webrev/

regards
Mark

--- old/test/java/net/CookieHandler/LocalHostCookie.java    Wed Aug 21
12:23:21 2013
+++ new/test/java/net/CookieHandler/LocalHostCookie.java    Wed Aug 21
12:23:21 2013
@@ -72,7 +72,9 @@
                  }
              }
          } finally {
-            s.stopServer();
+            if (s != null) {
+                s.stopServer();
+            }
          }
      }

@@ -96,7 +98,9 @@
          }

          public void stopServer() {
-            server.stop(0);
+            if (server != null) {
+                server.stop(0);
+            }
          }
      }