Fix the no note and text in metronome marks; also add regtest

diff --git a/input/regression/metronome-text.ly b/input/regression/metronome-text.ly
index ec771c57babe1f047be5b82764c8eb42d9e22b2a..8282dbcc3cdf995f30c73ced9d34bc6c751a0f05 100644 (file)
--- a/input/regression/metronome-text.ly
+++ b/input/regression/metronome-text.ly
@@ -23,6 +23,8 @@ Score.hideTempoNote, one can hide the duration=count in the tempo mark.
   \set Score.tempoHideNote = ##t
   \tempo "No note" 8=160 c1
   \tempo "Still not" c1
+  % No text and also no note => \null markup
+  \tempo 4=100 c1
   \tempo "Allegro" 4=120 c1
   \set Score.tempoHideNote = ##f
   \tempo "With note" 8=80 c1\break

diff --git a/scm/translation-functions.scm b/scm/translation-functions.scm
index 2e4b8a63049c0b72683f0d519b39edc0e5c8207c..3767a18be2be9de99549f3541ef00ea2b494dd66 100644 (file)
--- a/scm/translation-functions.scm
+++ b/scm/translation-functions.scm
@@ -21,25 +21,20 @@
                           (make-simple-markup  "=")
                           (make-simple-markup " ")
                           (make-simple-markup (number->string count))))
-                      (make-null-markup)))
+                      #f))
          (text-markup (if (not (null? text))
                         (make-bold-markup text)
                         #f)))
     (if text-markup
-      (if note-markup
+      (if (and note-markup (not hide-note))
         (make-line-markup (list text-markup
           (make-concat-markup (list (make-simple-markup "(")
                                     note-markup
                                     (make-simple-markup ")")))))
-        (make-line-markup (list text-markup))
-      )
+        (make-line-markup (list text-markup)))
       (if note-markup
         (make-line-markup (list note-markup))
-        #f
-      )
-    )
-  )
-)
+        (make-null-markup)))))
 
 (define-public (format-mark-alphabet mark context)
   (make-bold-markup (make-markalphabet-markup (1- mark))))