Fixes issue 1466 (intelligent linking erroneously stripping )s)

git-svn-id: http://svn.dmdirc.com/trunk@4384 00569f92-eb28-0410-84fd-f71c24880f

src/com/dmdirc/ui/messages/Styliser.java

@@ -79,13 +79,13 @@ public final class Styliser {
     private static final String URL_PUNCT_ILLEGAL = "\"";
     
     /** Defines all characters treated as trailing punctuation that're legal in URLs. */
-    private static final String URL_PUNCT_LEGAL = "';:!,\\.\\)\\?";
+    private static final String URL_PUNCT_LEGAL = "';:!,\\.\\?";
     
     /** Defines all trailing punctuation. */
     private static final String URL_PUNCT = URL_PUNCT_ILLEGAL + URL_PUNCT_LEGAL;
 
     /** Defines all characters allowed in URLs that aren't treated as trailing punct. */
-    private static final String URL_NOPUNCT = "a-z0-9$\\-_@&\\+\\*\\(=/#%~";
+    private static final String URL_NOPUNCT = "a-z0-9$\\-_@&\\+\\*\\(\\)=/#%~";
 
     /** Defines all characters allowed in URLs per W3C specs. */
     private static final String URL_CHARS = "[" + URL_PUNCT_LEGAL + URL_NOPUNCT

test/com/dmdirc/ui/messages/IntelligentLinkingTest.java

@@ -75,8 +75,8 @@ public class IntelligentLinkingTest {
             {"svn+ssh://foo@bar", "~svn+ssh://foo@bar~"},
             {"/var/web/www.foo.com/bar", "/var/web/www.foo.com/bar"},
             {"\"foo\" www.google.com \"www.google.com\"",
-                     "\"foo\" ~www.google.com~ \"~www.google.com~\"",
-            },
+                     "\"foo\" ~www.google.com~ \"~www.google.com~\""},
+            {"www.example.com/blah(foobar)", "~www.example.com/blah(foobar)~"},
         };
 
         return Arrays.asList(tests);