Re: [evolution-patches] crash at startup, 41448



On Wed, 2003-04-23 at 03:44, Jeffrey Stedfast wrote:
> Looks good. Probably could do without the g_malloc0() in
> camel_url_copy() since all the fields are immediately set. The only one
> you would have to init to NULL would be the new->params field.

Yeah,i thought of that at the time, and thought that if the struct ever
gets more fields, the copy function will probably be forgotten, so its
the easiest way to make it degrade gracefully.  Though then it just
hides bugs ...

i'll remove it, and commit





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