Nicolas Sceaux wrote:
Erlend Aasland <[EMAIL PROTECTED]> writes:


+(defmacro-public def-format-mark-command (type cmd)
+  (let* ((pfx "format-mark-")
+         (sfx (symbol->string type))
+         (ord-cmd (string->symbol (string-append pfx sfx)))
+         (box-cmd (string->symbol (string-append pfx (string-append "box-" 
sfx))))
+         (circle-cmd (string->symbol (string-append pfx (string-append 
"circle-" sfx)))))
+  `(begin
+    (define-public (,ord-cmd mark context) (make-bold-markup (,@cmd)))
+    (define-public (,box-cmd mark context) (make-bold-markup (make-box-markup 
(,@cmd))))
+    (define-public (,circle-cmd mark context) (make-bold-markup 
(make-circle-markup (,@cmd))))
+)))


It would be good if `def-format-mark-command' had a detailed
documentation string, especially as it creates symbols to name new
functions.

I actually think that this is not a clean up. The original, while lengthy, was perfectly easy to understand. I don't why you need to add (a public!) macro to condense it further. Maybe we should think of a better interface to the mark formatter. MAybe we could do something like

  markMarkup = \markup  {
    \circle \bold \fromproperty #'context:bar-number
  }

and drop the special markFormatter interface.

Any  takers / sponsors for a patch?

--
 Han-Wen Nienhuys - [EMAIL PROTECTED] - http://www.xs4all.nl/~hanwen


_______________________________________________
lilypond-devel mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/lilypond-devel

Reply via email to