Fix some bugs in the dynamic engraver and PostScript backend

[lilypond.git] / lily / mark-engraver.cc

diff --git a/lily/mark-engraver.cc b/lily/mark-engraver.cc
index 2715cf8ad036f8346b5cdf238c029373326da276..cb2ac141d9e975ec9a18e3f5677394b407a77fd2 100644 (file)
--- a/lily/mark-engraver.cc
+++ b/lily/mark-engraver.cc
@@ -11,13 +11,14 @@ using namespace std;
 
 #include "engraver.hh"
 
+#include "axis-group-interface.hh"
 #include "bar-line.hh"
 #include "context.hh"
+#include "grob-array.hh"
+#include "international.hh"
 #include "item.hh"
-#include "warn.hh"
 #include "text-interface.hh"
-#include "grob-array.hh"
-#include "axis-group-interface.hh"
+#include "warn.hh"
 
 /**
    put stuff over or next to  bars.  Examples: bar numbers, marginal notes,
@@ -38,6 +39,7 @@ protected:
   void process_music ();
   void stop_translation_timestep ();
 
+  DECLARE_ACKNOWLEDGER (break_alignment);
   DECLARE_ACKNOWLEDGER (break_aligned);
 };
 
@@ -47,13 +49,20 @@ Mark_engraver::Mark_engraver ()
   mark_ev_ = 0;
 }
 
+/*
+  This is a flawed approach, since various break-aligned objects may
+  not appear depending on key signature etc.
+
+   We keep it in case someone puts the engraver in a lower context than score.
+ */
 void
 Mark_engraver::acknowledge_break_aligned (Grob_info inf)
 {
   Grob *s = inf.grob ();
   if (text_
-      && (get_property ("rehearsalMarkAlignSymbol")
-         == s->get_property ("break-align-symbol"))
+      && !text_->get_parent (X_AXIS)
+      && (text_->get_property_data (ly_symbol2scm ("break-align-symbol"))
+         == s->get_property_data (ly_symbol2scm ("break-align-symbol")))
       && Axis_group_interface::has_interface (s))
     {
       /*
@@ -64,6 +73,18 @@ Mark_engraver::acknowledge_break_aligned (Grob_info inf)
     }
 }
 
+void
+Mark_engraver::acknowledge_break_alignment (Grob_info inf)
+{
+  Grob *s = inf.grob ();
+  if (text_
+      && dynamic_cast<Item *> (s))
+    {
+      text_->set_parent (s, X_AXIS);
+    }
+}
+
+
 void
 Mark_engraver::stop_translation_timestep ()
 {
@@ -140,6 +161,7 @@ Mark_engraver::process_music ()
 #include "translator.icc"
 
 ADD_ACKNOWLEDGER (Mark_engraver, break_aligned);
+ADD_ACKNOWLEDGER (Mark_engraver, break_alignment);
 
 ADD_TRANSLATOR (Mark_engraver,
                /* doc */ "This engraver will create RehearsalMark objects. "
@@ -148,7 +170,12 @@ ADD_TRANSLATOR (Mark_engraver,
                "to a different context, "
                "@ref{Staff_collecting_engraver} must move along, otherwise all marks"
                "end up on the same Y-location",
+               
                /* create */ "RehearsalMark",
                /* accept */ "mark-event",
-               /* read */ "rehearsalMark rehearsalMarkAlignSymbol markFormatter stavesFound",
+               /* read */
+               "markFormatter "
+               "rehearsalMark "
+               "stavesFound ",
+               
                /* write */ "");