Re: gEDA-dev: pcb CVS

["Bernd Jendrissek" <bernd_dot_jendrissek-at-NOSPAMgmail_dot_com>]

On Jan 11, 2008 10:23 PM, Larry Doolittle <ldoolitt@recycle.lbl.gov> wrote:
> --- /home/larry/cvs/pcb/src/polygon1.c  2008-01-11 09:46:24.000000000 -0800
> +++ pcb/src/polygon1.c  2008-01-11 10:18:07.000000000 -0800
> @@ -1947,9 +1947,10 @@
>      return;
>    for (cur = (*c)->head.prev; cur != &(*c)->head; free (cur->next))
>      cur = cur->prev;
> -  /* FIXME -- strict aliasing violation.  */
> -  if ((*c)->tree)
> -    r_destroy_tree ((rtree_t **) & ((*c)->tree));
> +  if ((*c)->tree) {
> +    rtree_t *r = (*c)->tree;
> +    r_destroy_tree (&r);
> +  }
>    free (*c), *c = NULL;
>
>  }

r_destroy_tree() will now set r to NULL, instead of (*c)->tree.
Luckily it's moot due to the following free(*c).


_______________________________________________
geda-dev mailing list
geda-dev@moria.seul.org
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev