Re: [evolution-patches] patch to enable 'copy email address' on right-click menu for email addresses



Em Qua, 2004-11-17 às 15:39 -0500, Jeffrey Stedfast escreveu:
> this should probably be implemented as an EPlugin. Also, don't use
> gchar, use char.

Will take a look at this.

> g_strsplit is also completely the wrong way to parse the string for
> several reasons:
> 
> 1. you're leaking memory

Yeah, indeed, but that's a problem in my code, and not something
inherent to g_strsplit, right?

> 2. mailto uri's can contain more than just the email address
> 3. the email addresses in the mailto uri can be encoded

gah, you're right, haven't thought about this... will take a look too.

Thanks for the guidance,

-- 
kov debian org: Gustavo Noronha <http://people.debian.org/~kov>
Debian:  <http://www.debian.org>  *  <http://www.debian-br.org>





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]