Hi Stuart,

Stuart Brorson writes:
 > [...]
 > As the ancient saying goes, be careful what you wish for.  I just
 > uploaded all the stuff to CVS.  Any interested developers, please
 > play with it!  If you notice any bugs, please make a detailed report,
 > saying what sequence of events caused the bug to occur.

I have reviewed your code and I have found some problems, mainly
memory related. I have only checked libgeda and gschem. A diff against
each of these packages is attached.

BTW you should take the opportunity to document your changes directly
in the texinfo parts of the noweb files. You should also avoid adding
C comments in these documentation parts (the /* --- */ lines before
the @sections).

I have not extensively tested, only reviewed the code.

Regards,


Patrick

Attachment: libgeda.diff
Description: Binary data

Attachment: gschem.diff
Description: Binary data

Reply via email to