* lily/bar-line.cc: Add documentation for "||:" also in the

[lilypond.git] / lily / paper-column-engraver.cc

diff --git a/lily/paper-column-engraver.cc b/lily/paper-column-engraver.cc
index 276834951175821cdd15b9f4e14c339c07a1b2f5..d3602d732a3b9c49b4c74ad036f72432197b88af 100644 (file)
--- a/lily/paper-column-engraver.cc
+++ b/lily/paper-column-engraver.cc
@@ -7,8 +7,6 @@
 */
 
 #include "paper-column-engraver.hh"
-#include "system.hh"
-#include "international.hh"
 #include "axis-group-interface.hh"
 #include "context.hh"
 #include "item.hh"
@@ -28,9 +26,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 +40,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_);
     }
 }
@@ -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)
 {
@@ -171,56 +167,6 @@ 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 ()
 {
@@ -246,50 +192,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 +247,9 @@ ADD_TRANSLATOR (Paper_column_engraver,
                /* accept */ "break-event",
                /* read */
                 "forbidBreak "
-                "allowPageTurn "
-               "revokePageTurns "
-               ,
+                "allowPageTurn",
                /* write */
                 "forbidBreak "
                 "allowPageTurn "
-               "revokePageTurns "
                "currentCommandColumn "
-               "currentMusicalColumn "
-               );
+               "currentMusicalColumn");