On Tue, 5 Aug 2008 14:40:03 +0200, "Bernd Jendrissek"
<[EMAIL PROTECTED]> wrote:
>
> [*] Examples of what I consider "just totally insane" and
> abort()-worthy would be toplevel == NULL, or o_current->type !=
> anything we know.
>

toplevel == NULL there's an argument for being totally insane.

**IF** we had opaque objects[1] then the latter is insane. For the time
being, because a client application might decide it's got a good reason to
create its own custom object types (goodness knows why) and we provide the
APIs required to do so (goodness knows why), we need to assume that it'll
happen.

I still think libgeda should be statically linked into the gEDA apps at
build time and not installed. :(


                             Peter


[1] Bernd, your opaque objects stuff is pretty cool. But there seems to be
so much random stuff mixed in with it! I really wish you'd use stgit or
something so that you could keep your patches in a sane order, and
immediately submit anything that no-one would dispute belongs on trunk. At
the moment, if we want to merge your humongous branch, it looks like we're
just going to have to take your word for it that it doesn't contain any
changes that are going to come back and bite us in the ass, because I don't
see how anyone's going to successfully review it.  D-:



_______________________________________________
geda-dev mailing list
[email protected]
http://www.seul.org/cgi-bin/mailman/listinfo/geda-dev

Reply via email to