|
From: Ethan M. <merritt@u.washington.edu> - 2006-07-14 00:09:31
|
On Thursday 13 July 2006 11:27 am, Juergen Wieferink wrote:
>
> Well, it could be done like the others:
>
> fill_gpval_string("GPVAL_TERM", term->name);
Fine.
> I'd suggest something like:
>
> Index: eval.c
> ===================================================================
> --- eval.c (Revision 279)
> +++ eval.c (Arbeitskopie)
> @@ -733,8 +733,10 @@
> return;
> if (v->udv_undef == FALSE && !strcmp((char*)&v->udv_value,
> value)) return;
> - v->udv_undef = FALSE;
> - gpfree_string(&v->udv_value);
> + if (v->udv_undef)
> + v->udv_undef = FALSE;
> + else
> + gpfree_string(&v->udv_value);
> Gstring(&v->udv_value, gp_strdup(value));
Agreed. Both now in cvs.
--
Ethan A Merritt
Biomolecular Structure Center
University of Washington, Seattle WA
|