Hi all,

I've just pushed a small patch to fix support for non ascii characters
in go-to feature. It concerns the uri_escape function. The small patch
(for discussion) follow.

I'm not sure if my fix is complete or not. Or if it is really portable
or not. Locale related bugs are not easy.
I think the best way is to check on many different configurations. Can
you give it an eye and test the fix? The test is quite simple: open
the 'Go to" dialog, select Nominatim service and search for a name
with non ascii characters.

Many thanks in advance.

diff --git a/src/util.c b/src/util.c
index 5c9823b..8b188d7 100644
--- a/src/util.c
+++ b/src/util.c
@@ -102,7 +102,7 @@ gchar *uri_escape(gchar *str)
     else if (g_ascii_isalnum(*src))
      *dst++ = *src;
     else {
-      g_sprintf(dst, "%%%02X", *src);
+      g_sprintf(dst, "%%%02hhX", *src);
       dst += 3;
     }
   }


-- 
Guilhem BONNEFILLE
-=- JID: gu...@im.apinc.org MSN: guilhem_bonnefi...@hotmail.com
-=- mailto:guilhem.bonnefi...@gmail.com
-=- http://nathguil.free.fr/

------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Viking-devel mailing list
Viking-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/viking-devel
Viking home page: http://viking.sf.net/

Reply via email to