commit 2b1078298c53135beafa6f7c5085cec41b10f151
author Gustaf Lundh <gustaf.lundh@sonyericsson.com> Mon Sep 12 12:51:38 2011 +0200
committer Gustaf Lundh <gustaf.lundh@sonyericsson.com> Mon Sep 12 12:52:54 2011 +0200
tree 89e15a1e07d0a7d0787d5f55a5e12b1c4f4af35a
parent 742a56e31dcab7bb3314c47107ac00e7e47e6464

Fix: Issue where Gerrit could not linkify certain URLs

In commit 742a56e31dcab7bb3314c47107ac00e7e47e6464
support was added to allow Gerrit to linkify URLs
containing "~".

However, since the "~"-character was added after the
dash "-" in the regex pattern:

   742a56: [a-zA-Z0-9$_.+!*',%;:@=?#/-~]
                                      ^
...it effectively changed the meaning of "-" from
a literal character to instead include the full range
of characters between "/" and "~" (EXCLUDING
the actual "-" sign)

This patch allows Gerrit to again acknowledge the "-"
to be a part of the url.

Change-Id: Ia7754481daea103be0664891aa612d6a13d8e344

src/main/java/com/google/gwtexpui/safehtml/client/SafeHtml.java

diff --git a/src/main/java/com/google/gwtexpui/safehtml/client/SafeHtml.java b/src/main/java/com/google/gwtexpui/safehtml/client/SafeHtml.java
index 32d9a78..42d76cc 100644
--- a/src/main/java/com/google/gwtexpui/safehtml/client/SafeHtml.java
+++ b/src/main/java/com/google/gwtexpui/safehtml/client/SafeHtml.java
@@ -111,7 +111,7 @@
   /** Convert bare http:// and https:// URLs into <a href> tags. */
   public SafeHtml linkify() {
     final String part = "(?:" +
-		"[a-zA-Z0-9$_.+!*',%;:@=?#/-~]" +
+		"[a-zA-Z0-9$_.+!*',%;:@=?#/~-]" +
 		"|&(?!lt;|gt;)" +
 		")";
     return replaceAll(