Re: [evolution-patches] patch for #42212



On Wed, 2003-05-07 at 18:22, Rodrigo Moya wrote:
> On Wed, 2003-05-07 at 18:30, JP Rosevear wrote:
> > If we're going to making passing a null string an invalid param, we need
> > to check everywhere that we don't pass a null param in.
> > 
> all callers of e_uri_new are expected to check the return value, since
> the URI might be an invalid one. A grep over the sources shows that all
> cases check for the return value.

Yes, but g_return_val_if_fail implies that a NULL uri string should
never be passed in at all.  If its allowed, then the check should be

if (!string)
    return;

or something similar.

-JP
-- 
JP Rosevear <jpr ximian com>
Ximian, Inc.




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