Fix some bugs in the dynamic engraver and PostScript backend

[lilypond.git] / lily / ledger-line-engraver.cc

diff --git a/lily/ledger-line-engraver.cc b/lily/ledger-line-engraver.cc
index 4ab91ece0fa7a5fd7172147c37bc68d1a19eae8f..550dc5a014ed95ec97da52e2ac4a6a96b7965050 100644 (file)
--- a/lily/ledger-line-engraver.cc
+++ b/lily/ledger-line-engraver.cc
@@ -3,7 +3,7 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 2004--2005 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  (c) 2004--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
 */
 
 #include "pointer-group-interface.hh"
@@ -20,10 +20,10 @@ public:
 
 protected:
   virtual void finalize ();
-  PRECOMPUTED_VIRTUAL void process_music ();
- 
-  DECLARE_ACKNOWLEDGER(ledgered);
-  DECLARE_ACKNOWLEDGER(staff_symbol);
+  void process_music ();
+
+  DECLARE_ACKNOWLEDGER (ledgered);
+  DECLARE_ACKNOWLEDGER (staff_symbol);
 
   void start_spanner ();
   void stop_spanner ();
@@ -46,6 +46,9 @@ Ledger_line_engraver::start_spanner ()
 void
 Ledger_line_engraver::process_music ()
 {
+  /*
+    Need to do this, otherwise the first note might miss ledgers. 
+  */
   if (!span_)
     start_spanner ();
 }
@@ -53,7 +56,7 @@ Ledger_line_engraver::process_music ()
 void
 Ledger_line_engraver::finalize ()
 {
-  stop_spanner();
+  stop_spanner ();
 }
 
 void
@@ -62,6 +65,7 @@ Ledger_line_engraver::stop_spanner ()
   if (span_)
     {
       span_->set_bound (RIGHT, unsmob_grob (get_property ("currentCommandColumn")));
+      Pointer_group_interface::set_ordered (span_, ly_symbol2scm ("elements"), false);
       span_ = 0;
     }
 }
@@ -69,10 +73,11 @@ Ledger_line_engraver::stop_spanner ()
 void
 Ledger_line_engraver::acknowledge_staff_symbol (Grob_info s)
 {
-  Spanner *sym = dynamic_cast<Spanner*> (s.grob ());
+  Spanner *sym = dynamic_cast<Spanner *> (s.grob ());
 
   if (!span_
-      || span_->get_bound (LEFT) != sym->get_bound (LEFT))
+      || (span_->get_bound (LEFT) != sym->get_bound (LEFT)
+         && sym->get_bound (LEFT)))
     {
       stop_spanner ();
       start_spanner ();
@@ -92,11 +97,11 @@ Ledger_line_engraver::acknowledge_ledgered (Grob_info s)
 
 #include "translator.icc"
 
-ADD_ACKNOWLEDGER(Ledger_line_engraver,ledgered);
-ADD_ACKNOWLEDGER(Ledger_line_engraver,staff_symbol);
+ADD_ACKNOWLEDGER (Ledger_line_engraver, ledgered);
+ADD_ACKNOWLEDGER (Ledger_line_engraver, staff_symbol);
 ADD_TRANSLATOR (Ledger_line_engraver,
                "Creates the spanner to draw ledger lines, and notices objects that need ledger lines",
-               /* creats*/ "LedgerLineSpanner",
-               /* accepts */ "",
-               /* reads */ "",
+               /* create */ "LedgerLineSpanner",
+               /* accept */ "",
+               /* read */ "",
                /* write */ "")