[frogs] Re: T1055: Avoid using deprecated %module-public-interface in guile initialisation. (issue1160044)

[ Thread Index | Date Index | More lilynet.net/frogs Archives ]


Hi Ian,

I haven't tested your patch yet to make sure everything is okay, but I
have a few quick comments for you.

Thanks,
Patrick


http://www.codereview.appspot.com/1160044/diff/10001/11008
File scm/lily.scm (right):

http://www.codereview.appspot.com/1160044/diff/10001/11008#newcode250
scm/lily.scm:250: (define-public  (guile-v2 )
(define-public (guile-v2)

http://www.codereview.appspot.com/1160044/diff/10001/11008#newcode251
scm/lily.scm:251: (string>? (version) "1.8.7"))
What if the Guile team decides to release 1.8.8 ?

It would be safer to change this to some 1.9 version...

http://www.codereview.appspot.com/1160044/diff/10001/11008#newcode303
scm/lily.scm:303: (cond-expand
Shouldn't this `cond-expand' block start in column 1, and then indented
from there?

http://www.codereview.appspot.com/1160044/show

---
----
Join the Frogs!


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/