]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/paper-column-engraver.cc
Fix some bugs in the dynamic engraver and PostScript backend
[lilypond.git] / lily / paper-column-engraver.cc
index 276834951175821cdd15b9f4e14c339c07a1b2f5..7bc7c665ac1798d978ee09c17e248965c2bf4952 100644 (file)
@@ -8,15 +8,14 @@
 
 #include "paper-column-engraver.hh"
 #include "system.hh"
-#include "international.hh"
-#include "axis-group-interface.hh"
-#include "context.hh"
 #include "item.hh"
-#include "note-spacing.hh"
 #include "paper-column.hh"
-#include "pointer-group-interface.hh"
 #include "staff-spacing.hh"
-#include "system.hh"
+#include "note-spacing.hh"
+#include "pointer-group-interface.hh"
+#include "context.hh"
+#include "score-context.hh"
+#include "axis-group-interface.hh"
 #include "warn.hh"
 
 #include "translator.icc"
@@ -28,9 +27,9 @@ Paper_column_engraver::Paper_column_engraver ()
   musical_column_ = 0;
   breaks_ = 0;
   system_ = 0;
-  last_special_barline_column_ = 0;
-  last_breakable_column_ = 0;
   first_ = true;
+  last_breakable_column_ = 0;
+  last_breakable_moment_ = Moment (-1);
 }
 
 void
@@ -42,7 +41,6 @@ Paper_column_engraver::finalize ()
   if (command_column_)
     {
       command_column_->set_property ("line-break-permission", ly_symbol2scm ("allow"));
-      command_column_->set_property ("page-turn-permission", ly_symbol2scm ("allow"));
       system_->set_bound (RIGHT, command_column_);
     }
 }
@@ -55,11 +53,10 @@ Paper_column_engraver::make_columns ()
   */
   Paper_column *p1 = make_paper_column ("NonMusicalPaperColumn");
   Paper_column *p2 = make_paper_column ("PaperColumn");
-  /* 
-     The columns are timestamped with now_mom () in
-     stop_translation_timestep. Cannot happen now, because the
-     first column is sometimes created before now_mom is initialised.
-  */
+
+  SCM m = now_mom ().smobbed_copy ();
+  p1->set_property ("when", m);
+  p2->set_property ("when", m);
 
   set_columns (p1, p2);
 }
@@ -87,7 +84,6 @@ Paper_column_engraver::acknowledge_staff_spacing (Grob_info gi)
                                     ly_symbol2scm ("spacing-wishes"),
                                     gi.grob ());
 }
-
 void
 Paper_column_engraver::acknowledge_note_spacing (Grob_info gi)
 {
@@ -112,11 +108,12 @@ Paper_column_engraver::set_columns (Paper_column *new_command,
   system_->add_column (musical_column_);
 }
 
-IMPLEMENT_TRANSLATOR_LISTENER (Paper_column_engraver, break);
-void
-Paper_column_engraver::listen_break (Stream_event *ev)
+bool
+Paper_column_engraver::try_music (Music *m)
 {
-  break_events_.push_back (ev);
+  break_events_.push_back (m);
+
+  return true;
 }
 
 void
@@ -125,17 +122,18 @@ Paper_column_engraver::process_music ()
   for (vsize i = 0; i < break_events_.size (); i++)
     {
       string prefix;
-      SCM name_sym = break_events_[i]->get_property ("class");
-      string name = ly_scm2string (scm_symbol_to_string (name_sym));
-      size_t end = name.rfind ("-event");
-      if (end)
-       prefix = name.substr (0, end);
+      SCM name = break_events_[i]->get_property ("name");
+      if (name == ly_symbol2scm ("LineBreakEvent"))
+       prefix = "line-break";
+      else if (name == ly_symbol2scm ("PageBreakEvent"))
+       prefix = "page-break";
+      else if (name == ly_symbol2scm ("PageTurnEvent"))
+       prefix = "page-turn";
       else
        {
          programming_error ("Paper_column_engraver doesn't know about this break-event");
          return;
        }
-
       string perm_str = prefix + "-permission";
       string pen_str = prefix + "-penalty";
 
@@ -171,63 +169,9 @@ Paper_column_engraver::process_music ()
     }
 }
 
-/* return either
-   - the last column with a special (ie. not "|" or "") barline
-   - the last column
-   after the given moment
-*/
-Paper_column*
-Paper_column_engraver::find_turnable_column (Moment after_this)
-{
-  if (last_special_barline_column_)
-    {
-      Moment m = *unsmob_moment (last_special_barline_column_->get_property ("when"));
-      if (m >= after_this)
-       return last_special_barline_column_;
-    }
-  if (last_breakable_column_)
-    {
-      Moment m = *unsmob_moment (last_breakable_column_->get_property ("when"));
-      if (m >= after_this)
-       return last_breakable_column_;
-    }
-  return 0;
-}
-
-void
-Paper_column_engraver::revoke_page_turns (Moment after_this, Real new_penalty)
-{
-  if (!page_turnable_columns_.size ())
-    return;
-
-  for (vsize i = page_turnable_columns_.size () - 1; i--;)
-    {
-      Paper_column *col = page_turnable_columns_[i];
-      Moment mom = *unsmob_moment (col->get_property ("when"));
-      if (mom >= after_this)
-       {
-         if (isinf (new_penalty))
-           {
-             col->del_property ( ly_symbol2scm ("page-turn-permission"));
-             page_turnable_columns_.erase (page_turnable_columns_.begin () + i);
-           }
-         else
-           {
-             Real prev_pen = robust_scm2double (col->get_property ("page-turn-penalty"), 0);
-             if (new_penalty > prev_pen)
-               col->set_property ("page-turn-penalty", scm_from_double (new_penalty));
-           }
-       }
-    }
-}
-
 void
 Paper_column_engraver::stop_translation_timestep ()
 {
-  SCM m = now_mom ().smobbed_copy ();
-  command_column_->set_property ("when", m);
-  musical_column_->set_property ("when", m);
-
   for (vsize i = 0; i < items_.size (); i++)
     {
       Item *elem = items_[i];
@@ -246,50 +190,25 @@ Paper_column_engraver::stop_translation_timestep ()
     {
       breaks_++;
       last_breakable_column_ = command_column_;
-
-      SCM which_bar = get_property ("whichBar");
-      if (scm_is_string (which_bar))
-       {
-         string bar = ly_scm2string (which_bar);
-         if (bar != "" && bar != "|")
-           last_special_barline_column_ = command_column_;
-       }
-
+      last_breakable_moment_ = now_mom ();
       if (! (breaks_%8))
        progress_indication ("[" + to_string (breaks_) + "]");
     }
 
   SCM page_br = get_property ("allowPageTurn");
-  if (scm_is_pair (page_br) && last_breakable_column_)
+  if (scm_is_pair (page_br) && last_breakable_moment_ >= Rational (0))
     {
       SCM pen = scm_cdr (page_br);
       Moment *m = unsmob_moment (scm_car (page_br));
-      if (m)
+      if (m && scm_is_number (pen) && *m <= last_breakable_moment_)
        {
-         Paper_column *turn = find_turnable_column (*m);
-         if (turn)
-           {
-             turn->set_property ("page-turn-permission", ly_symbol2scm ("allow"));
-             turn->set_property ("page-turn-penalty", pen);
-             page_turnable_columns_.push_back (turn);
-           }
+         last_breakable_column_->set_property ("page-turn-permission", ly_symbol2scm ("allow"));
+         last_breakable_column_->set_property ("page-turn-penalty", pen);
        }
     }
 
-  /* The page-turn-engraver is allowed to change its mind and revoke previously-allowed
-     page turns (for example if there is a volta repeat where a turn is inconvenient) */
-  SCM revokes = get_property ("revokePageTurns");
-  if (scm_is_pair (revokes))
-    {
-      Moment *start = unsmob_moment (scm_car (revokes));
-      Real pen = robust_scm2double (scm_cdr (revokes), infinity_f);
-      if (start)
-       revoke_page_turns (*start, pen);
-    }
-
   context ()->get_score_context ()->unset_property (ly_symbol2scm ("forbidBreak"));
   context ()->get_score_context ()->unset_property (ly_symbol2scm ("allowPageTurn"));
-  context ()->get_score_context ()->unset_property (ly_symbol2scm ("revokePageTurns"));
 
   first_ = false;
   break_events_.clear ();
@@ -326,13 +245,9 @@ ADD_TRANSLATOR (Paper_column_engraver,
                /* accept */ "break-event",
                /* read */
                 "forbidBreak "
-                "allowPageTurn "
-               "revokePageTurns "
-               ,
+                "allowPageTurn",
                /* write */
                 "forbidBreak "
                 "allowPageTurn "
-               "revokePageTurns "
                "currentCommandColumn "
-               "currentMusicalColumn "
-               );
+               "currentMusicalColumn");