X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fledger-line-spanner.cc;h=3664bd40b7cb4c700467758903d060fb7ebef59f;hb=974b5c76bc9606752de44d128c13a9f2adc0e322;hp=43bc35b69596a01107fbb5a11b86ab1683466913;hpb=7e33691750a97e1b66192cb484f479a173390132;p=lilypond.git diff --git a/lily/ledger-line-spanner.cc b/lily/ledger-line-spanner.cc index 43bc35b695..3664bd40b7 100644 --- a/lily/ledger-line-spanner.cc +++ b/lily/ledger-line-spanner.cc @@ -3,13 +3,12 @@ source file of the GNU LilyPond music typesetter - (c) 2004--2006 Han-Wen Nienhuys + (c) 2004--2007 Han-Wen Nienhuys */ #include using namespace std; -#include "item.hh" #include "note-head.hh" #include "staff-symbol-referencer.hh" #include "staff-symbol.hh" @@ -29,7 +28,7 @@ struct Ledger_line_spanner Interval x_extent, Real left_shorten); - static bool has_interface (Grob *); + DECLARE_GROB_INTERFACE (); }; Stencil @@ -43,7 +42,7 @@ Ledger_line_spanner::brew_ledger_lines (Grob *staff, { int line_count = (staff_extent.contains (pos) ? 0 - : sign (pos) * int (rint(pos - staff_extent[Direction (sign (pos))])) / 2); + : sign (pos) * int (rint (pos - staff_extent[Direction (sign (pos))])) / 2); Stencil stencil; if (line_count) { @@ -119,7 +118,10 @@ Ledger_line_spanner::set_spacing_rods (SCM smob) // find size of note heads. Grob *staff = Staff_symbol_referencer::get_staff_symbol (me); if (!staff) - return SCM_EOL; + { + me->suicide (); + return SCM_EOL; + } Real min_length_fraction = robust_scm2double (me->get_property ("minimum-length-fraction"), 0.15); @@ -139,7 +141,7 @@ Ledger_line_spanner::set_spacing_rods (SCM smob) contain a lot of noteheads, superlinear performance is too slow. */ extract_item_set (me, "note-heads", heads); - for (int i = heads.size (); i--;) + for (vsize i = heads.size (); i--;) { Item *h = heads[i]; @@ -206,7 +208,7 @@ Ledger_line_spanner::print (SCM smob) extract_grob_set (me, "note-heads", heads); - if (heads.is_empty ()) + if (heads.empty ()) return SCM_EOL; // find size of note heads. @@ -231,13 +233,13 @@ Ledger_line_spanner::print (SCM smob) { Axis a = Axis (i); common[a] = common_refpoint_of_array (heads, me, a); - for (int i = heads.size (); i--;) + for (vsize i = heads.size (); i--;) if (Grob *g = unsmob_grob (me->get_object ("accidental-grob"))) common[a] = common[a]->common_refpoint (g, a); } Ledger_requests reqs; - for (int i = heads.size (); i--;) + for (vsize i = heads.size (); i--;) { Item *h = dynamic_cast (heads[i]); @@ -290,7 +292,7 @@ Ledger_line_spanner::print (SCM smob) && !staff_extent.contains (i->second[d].position_ - sign (i->second[d].position_))); Real limit = (center + (both ? which * gap / 2 : 0)); - lr.ledger_extent_.elem_ref (-which) + lr.ledger_extent_.at (-which) = which * max (which * lr.ledger_extent_[-which], which * limit); } while (flip (&which) != LEFT); @@ -302,7 +304,7 @@ Ledger_line_spanner::print (SCM smob) // create ledgers for note heads Real ledgerlinethickness = Staff_symbol::get_ledger_line_thickness (staff); - for (int i = heads.size (); i--;) + for (vsize i = heads.size (); i--;) { Item *h = dynamic_cast (heads[i]); @@ -349,28 +351,27 @@ Ledger_line_spanner::print (SCM smob) } ADD_INTERFACE (Ledger_line_spanner, - "ledger-line-interface", - - "This spanner draws the ledger lines of a staff.\n" - "This is a separate grob because it has to process\n" - "all potential collisions between all note heads.", + "This spanner draws the ledger lines of a staff. This is a" + " separate grob because it has to process all potential" + " collisions between all note heads.", /* properties */ + "gap " + "length-fraction " + "minimum-length-fraction " "note-heads " "thickness " - "minimum-length-fraction " - "length-fraction " - "gap"); + ); struct Ledgered_interface { - static bool has_interface (Grob *); + DECLARE_GROB_INTERFACE (); }; ADD_INTERFACE (Ledgered_interface, - "ledgered-interface", + "Objects that need ledger lines, typically note heads. See" + " also @ref{ledger-line-spanner-interface}.", - "Objects that need ledger lines, typically " - "note heads. See also @ref{ledger-line-interface}.", - - "no-ledgers"); + /* properties */ + "no-ledgers " + );