Merge pull request #33 from Sidnicious/patch-1

Don't double-encode URLs before parsing them
b4winckler · Sep 6, 2013 · 0f3e96b · 0f3e96b
2 parents 52f4da8 + fe5039c
commit 0f3e96b
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/src/MacVim/MMAppController.m b/src/MacVim/MMAppController.m
@@ -1659,12 +1659,9 @@ - (void)handleXcodeModEvent:(NSAppleEventDescriptor *)event
 - (void)handleGetURLEvent:(NSAppleEventDescriptor *)event
                replyEvent:(NSAppleEventDescriptor *)reply
 {
-    NSString *urlString = [[event paramDescriptorForKeyword:keyDirectObject]
-        stringValue];
-    // NOTE: URLWithString requires string to be percent escaped.
-    urlString = [urlString stringByAddingPercentEscapesUsingEncoding:
-                                                        NSUTF8StringEncoding];
-    NSURL *url = [NSURL URLWithString:urlString];
+    NSURL *url = [NSURL URLWithString:[[event
+                                        paramDescriptorForKeyword:keyDirectObject]
+                                        stringValue]];
 
     // We try to be compatible with TextMate's URL scheme here, as documented
     // at http://blog.macromates.com/2007/the-textmate-url-scheme/ . Currently,