Re: Graphics code small questions?

[Michael Goffioul]

On 1/14/08, John W. Eaton <address@hidden> wrote:
> | virtual void remove_child (const graphics_handle& h)
> | {
> |   if (valid_object ())
> |     get_properties ().remove_child (h);
> |   else
> |     error ("base_graphics_object::remove_child: invalid graphics object");
> | }
> |
> | What do you think about it?
>
> I think that change is fine.  I think the reason it was done this way
> originally is that at first we did not have a get_properties method,
> and I needed some way to make the dispatch work.  I remember being
> slightly annoyed that I was duplicating these functions, but at the
> time I didn't see another way to do it.

OK, I'll prepare a patch when a get some time. In the meantime, if
anybody is willing to do it, please go ahead (but just tell me, so I don't
waste my time).

Michael.