Re: [evolution-patches] A NULL printf patch was committed
- From: Parthasarathi Susarla <sparthasarathi novell com>
- To: Harry Lu <Harry Lu Sun COM>
- Cc: patches <evolution-patches gnome org>
- Subject: Re: [evolution-patches] A NULL printf patch was committed
- Date: Tue, 29 Nov 2005 21:22:51 +0530
Hey Harry,
Thanks for the patch.
But i would suggest removing the printf() altogether. And do the
following instead:
[snip]
if (address_uri && (strncmp (address_uri, "mailto:", 7) == 0))
[snip]
Cheers,
partha
On Tue, 2005-11-29 at 19:17 +0800, Harry Lu wrote:
> - printf("get selection, address is '%s'\n", address_uri);
> -
> if (address_uri == NULL)
> return;
> +
> + printf("get selection, address is '%s'\n", address_uri);
>
> if (strncmp (address_uri, "mailto:", 7) == 0) {
> CamelInternetAddress *cia =
> camel_internet_address_new();
>
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]