[Xfce4-commits] r30446 - libexo/trunk/exo

[Nick Schermer]

Author: nick
Date: 2009-08-07 14:37:14 +0000 (Fri, 07 Aug 2009)
New Revision: 30446

Modified:
   libexo/trunk/exo/exo-url.c
Log:
Do not escape commas in url's (bug #5654).


Modified: libexo/trunk/exo/exo-url.c
===================================================================
--- libexo/trunk/exo/exo-url.c	2009-08-07 14:31:52 UTC (rev 30445)
+++ libexo/trunk/exo/exo-url.c	2009-08-07 14:37:14 UTC (rev 30446)
@@ -202,7 +202,6 @@
 {
   const gchar *category = NULL;
   gboolean     result = FALSE;
-  GString     *escaped_url;
   gchar       *display_name;
   gchar       *local_path;
   gchar       *command;
@@ -332,22 +331,8 @@
       return result;
     }
 
-  /* need to escape commata, otherwise firefox and several other helpers cannot handle the URL,
-   * see http://bugzilla.xfce.org/show_bug.cgi?id=2454 for a description of the problem.
-   */
-  escaped_url = g_string_sized_new (128);
-  for (; *url != '\0'; ++url)
-    {
-      if (*url == ',')
-        g_string_append (escaped_url, "%2C");
-      else
-        g_string_append_c (escaped_url, *url);
-    }
-
   /* oki doki then, let's open it */
-  result = exo_execute_preferred_application_on_screen (category, escaped_url->str, NULL, envp, screen, error);
-  g_string_free (escaped_url, TRUE);
-  return result;
+  return exo_execute_preferred_application_on_screen (category, url, NULL, envp, screen, error);
 }