gEDA

Comment 11 for bug 702752

Revision history for this message

Peter TB Brett (peter-b) wrote on 2011-06-13:

#11

I've split this into two patches (unrelated comments and the actual change), and I'm going to commit them.

It's not stylistically very good Scheme, I'm afraid. ;-) For example, this:

(if (null? format)
(set! format (gnetlist:get-package-attribute refdes "net-format"))
(set! format (car format)))

Would be better written as:

(set! format
      (if (null? format)
        (gnetlist:get-package-attribute refdes "net-format")
        (car format)))

Thanks for the patch.