[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: Calling all beta testers! Re: gEDA-user: Check for same refdes
- To: geda-user@xxxxxxxx
- Subject: Re: Calling all beta testers! Re: gEDA-user: Check for same refdes
- From: Ales Hvezda <ahvezda@xxxxxxxxxxxxxx>
- Date: Sat, 05 Feb 2005 18:28:53 -0500
- Delivered-to: archiver@seul.org
- Delivered-to: geda-user-outgoing@seul.org
- Delivered-to: geda-user@seul.org
- Delivery-date: Sat, 05 Feb 2005 18:29:17 -0500
- In-reply-to: Your message of "Sun, 23 Jan 2005 18:51:55 +0100." <16883.58427.19252.933669@mail.wanadoo.fr> ------
- Reply-to: geda-user@xxxxxxxx
- Sender: owner-geda-user@xxxxxxxx
Hi Patrick,
[snip]
>I think it would be even better if we can get ride of the s_scratch_*
>stuff (from libgeda). There is no real value added by these functions
>that build an array of strings unique or not. It is also memory hungry
>and last but not least it is based on a nasty hack.
Agreed. Getting rid of s_scratch_* is a very good idea.
I have looked over your patch and it superficially looks fine. Please
feel free to check it in. I will test gnetlist and make sure it works
after the checkin (you can also run the gnetlist regression tests in
gnetlist/tests by running: make tests)
>
>BTW, get_selected_component_attributes() and get_selected_filename()
>have very complicated prototypes: second and third arguments are never
>used and first may already be of type TOPLEVEL*. Any particular reason
>for that (except dependency on now dead code)?
Probably based on legacy code. If these functions can be
simplified, without breaking things, I'm all for it.
get-selected-component-attributes seems to be used in
gschem/scheme/generate_netlist.scm which was some work done for
gnetlist/examples/vams/*
-Ales