Re: [Spice-devel] [PATCH spice-gtk (v2)] Fix multiple problems with URI parsing

2012-04-19 Thread Daniel P. Berrange
On Thu, Apr 19, 2012 at 05:22:04PM +0200, Marc-André Lureau wrote: > On Thu, Apr 19, 2012 at 4:51 PM, Daniel P. Berrange > wrote: > > +    g_printerr("[%s][%s][%s][%s]\n", uri, authority, path, query); > > Is this a leftover? If not perhaps SPICE_DEBUG is more appropriate Doh. Yes it should not

Re: [Spice-devel] [PATCH spice-gtk (v2)] Fix multiple problems with URI parsing

2012-04-19 Thread Marc-André Lureau
On Thu, Apr 19, 2012 at 4:51 PM, Daniel P. Berrange wrote: > +    g_printerr("[%s][%s][%s][%s]\n", uri, authority, path, query); Is this a leftover? If not perhaps SPICE_DEBUG is more appropriate -- Marc-André Lureau ___ Spice-devel mailing list Spice

[Spice-devel] [PATCH spice-gtk (v2)] Fix multiple problems with URI parsing

2012-04-19 Thread Daniel P. Berrange
From: "Daniel P. Berrange" The URI parsing was not correctly skipping over the path component if a port number was specified using the traditional URI scheme, eg spice://somehost:5900/ would result in failed parse due to the trailing '/' The URI parsing was also not considering that the au