lilypond-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Reformat some files to let emacs do its work correctly. (issue 98630


From: Marc Hohl
Subject: Re: Reformat some files to let emacs do its work correctly. (issue 9863047)
Date: Fri, 07 Jun 2013 12:07:36 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130510 Thunderbird/17.0.6

Am 07.06.2013 09:07, schrieb Keith OHara:
On Thu, 06 Jun 2013 23:04:29 -0700, <address@hidden> wrote:

I think I'll draw a patch that corrects the minor faults; ideally,
running
fixscm again should lead in no changes at all.


That makes good sense.

It is hard to spot unwanted changes among the good changes from running
the script.


time-signature-settings.scm, maybe some newlines

@@ -312,24 +312,24 @@ a fresh copy of the list-head is made."
           (den (car revargs))
           (nums (reverse (cdr revargs))))
      (make-override-markup '(baseline-skip . 0)
-      (make-number-markup
-        (make-left-column-markup (list
-          (make-center-column-markup (list
-            (make-line-markup (insert-markups nums "+"))
-            den))))))))
+                          (make-number-markup
+                           (make-left-column-markup (list
+
(make-center-column-markup (list
+
(make-line-markup (insert-markups nums "+"))
+
den))))))))

done.

define-markup-commands.scm, maybe some ;;

3037 (define-public (horizontal-slash-interval num forward
number-interval mag)
3038   (if forward
3039       (cond ;((= num 6) (interval-widen number-interval (* mag 0.5)))
3040                                         ;((= num 5) (interval-widen
number-interval (* mag 0.5)))
...

done. Thanks!

BTW, I raised a new issue for this:

http://code.google.com/p/lilypond/issues/detail?id=3404

and uploaded a patch:

http://codereview.appspot.com/9964049

Regards,

Marc








reply via email to

[Prev in Thread] Current Thread [Next in Thread]