On Friday 25 July 2008 16:52:24 Patrick Bernaud wrote:
> Hello all,
>
> Again no new features. Only cleanup, renames and moves.
>
> Regards,
>
> Patrick
>

Thanks!

Just a note -- try to check that your patches don't add trailing whitespace.

Also, a couple of questions:

  06/17 Modify o_attrib_toggle_visibility() to work on a single object.
  07/17 Modify o_attrib_toggle_show_name_value() to work on a single object.

    What's the reasoning for this? The patch seems to add a lot of code. Is
    the idea to clean up the API? (I don't object to these changes, just
    wanted to check the rationale).

  14/17 Replace TILE_LOC structure with list of references to TILEs.

    What, if any, effect does this have on performance?

These patches look good. Thanks for all your hard work.

Cheers,

                                   Peter

-- 
Peter Brett

Electronic Systems Engineer
Integral Informatics Ltd

Attachment: signature.asc
Description: This is a digitally signed message part.


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

Reply via email to