Re: [evolution-patches] Patch to allow a contact to be opened in the editor from command-line.

[Not Zed <notzed ximian com>]

On Tue, 2005-07-12 at 13:22 +0530, Veerapuram Varadhan wrote:
> On Tue, 2005-07-12 at 09:06 +0800, Not Zed wrote:
> 
> > Easily.
> > 
> > url = url_new();
> > url.set_parameter("name", "value");
> > 
> > etc.
> > 
> > NEVER.  And I mean NEVER, EVER, use printf to create url strings.
> 
> IIUC, the above sample code will go in to the application that invokes
> evolution with a URI, isn't it?

Umm, the code in question was creating a uri string.  Thats all i was
talking about.

> And just do url.get_parameter () in handleURI code.  Did I get it right?
> 
> > 
> > How can it possibly do that? If you have a '&' encoded in the parameter
> > value, it will have to be %XX encoded separately to the &
> > 
> IINW, e_uri_new () in libedataserver/e-url.c does take care of decoding
> parameter values properly.

1. you weren't using e_uri anyway, and 2. the only code i saw, which i
was commenting about, was using query parameters, not uri parameters,
and it wasn't doing any encoding on them.