X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Fmark-engraver.cc;h=6eb7ee321de0054f6838d79f1725b81d4e229f6c;hb=47db9a3883d726ca53e2133a3b2298f78dd6a32e;hp=897475e6af103c11a9c8f46d3abeb91a546cbfea;hpb=4ecdbd7d70ca7441be4dddd15ac01cc255bc2a35;p=lilypond.git diff --git a/lily/mark-engraver.cc b/lily/mark-engraver.cc index 897475e6af..6eb7ee321d 100644 --- a/lily/mark-engraver.cc +++ b/lily/mark-engraver.cc @@ -1,21 +1,37 @@ /* - mark-engraver.cc -- implement Mark_engraver + This file is part of LilyPond, the GNU music typesetter. - source file of the GNU LilyPond music typesetter + Copyright (C) 1998--2015 Jan Nieuwenhuizen - (c) 1998--2005 Jan Nieuwenhuizen + LilyPond is free software: you can redistribute it and/or modify + it under the terms of the GNU General Public License as published by + the Free Software Foundation, either version 3 of the License, or + (at your option) any later version. + + LilyPond is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + GNU General Public License for more details. + + You should have received a copy of the GNU General Public License + along with LilyPond. If not, see . */ #include using namespace std; -#include "bar-line.hh" +#include "engraver.hh" + +#include "axis-group-interface.hh" #include "context.hh" -#include "engraver-group.hh" +#include "grob-array.hh" +#include "international.hh" #include "item.hh" -#include "warn.hh" +#include "stream-event.hh" #include "text-interface.hh" -#include "grob-array.hh" +#include "warn.hh" + +#include "translator.icc" /** put stuff over or next to bars. Examples: bar numbers, marginal notes, @@ -23,40 +39,45 @@ using namespace std; */ class Mark_engraver : public Engraver { + + void create_items (Stream_event *); + Item *text_; + Item *final_text_; + Stream_event *mark_ev_; + public: TRANSLATOR_DECLARATIONS (Mark_engraver); + protected: - Item *text_; -protected: - void stop_translation_timestep (); - DECLARE_ACKNOWLEDGER (bar_line); - void create_items (Music *); - virtual bool try_music (Music *ev); void process_music (); + void start_translation_timestep (); + void stop_translation_timestep (); + virtual void finalize (); -private: - Music *mark_ev_; + DECLARE_TRANSLATOR_LISTENER (mark); + DECLARE_ACKNOWLEDGER (break_alignment); }; Mark_engraver::Mark_engraver () { text_ = 0; + final_text_ = 0; mark_ev_ = 0; } void -Mark_engraver::acknowledge_bar_line (Grob_info inf) +Mark_engraver::acknowledge_break_alignment (Grob_info inf) { Grob *s = inf.grob (); - if (text_) - { - /* - TODO: make this configurable. RehearsalMark cannot be - break-aligned, since the width of the object should not be taken - into alignment considerations. - */ - text_->set_parent (s, X_AXIS); - } + if (text_ + && dynamic_cast (s)) + text_->set_parent (s, X_AXIS); +} + +void +Mark_engraver::start_translation_timestep () +{ + final_text_ = 0; } void @@ -65,14 +86,24 @@ Mark_engraver::stop_translation_timestep () if (text_) { text_->set_object ("side-support-elements", - grob_list_to_grob_array (get_property ("stavesFound"))); + grob_list_to_grob_array (get_property ("stavesFound"))); + final_text_ = text_; text_ = 0; } mark_ev_ = 0; } void -Mark_engraver::create_items (Music *ev) +Mark_engraver::finalize () +{ + if (final_text_) + final_text_->set_property ("break-visibility", + scm_c_make_vector (3, SCM_BOOL_T)); + final_text_ = 0; +} + +void +Mark_engraver::create_items (Stream_event *ev) { if (text_) return; @@ -80,11 +111,11 @@ Mark_engraver::create_items (Music *ev) text_ = make_item ("RehearsalMark", ev->self_scm ()); } -bool -Mark_engraver::try_music (Music *r) +IMPLEMENT_TRANSLATOR_LISTENER (Mark_engraver, mark); +void +Mark_engraver::listen_mark (Stream_event *ev) { - mark_ev_ = r; - return true; + ASSIGN_EVENT_ONCE (mark_ev_, ev); } /* @@ -98,51 +129,58 @@ Mark_engraver::process_music () create_items (mark_ev_); /* - automatic marks. + automatic marks. */ SCM m = mark_ev_->get_property ("label"); SCM proc = get_property ("markFormatter"); if (!Text_interface::is_markup (m) - && ly_is_procedure (proc)) - { - if (!scm_is_number (m)) - m = get_property ("rehearsalMark"); - - if (scm_integer_p (m) == SCM_BOOL_T - && scm_exact_p (m) == SCM_BOOL_T) - { - int mark_count = scm_to_int (m); - mark_count++; - context ()->set_property ("rehearsalMark", - scm_from_int (mark_count)); - } - - if (scm_is_number (m)) - m = scm_call_2 (proc, m, context ()->self_scm ()); - else - /* FIXME: constant error message. */ - warning (_ ("rehearsalMark must have integer value")); - } + && ly_is_procedure (proc)) + { + if (!scm_is_number (m)) + m = get_property ("rehearsalMark"); + + if (scm_integer_p (m) == SCM_BOOL_T + && scm_exact_p (m) == SCM_BOOL_T) + { + int mark_count = scm_to_int (m); + mark_count++; + context ()->set_property ("rehearsalMark", + scm_from_int (mark_count)); + } + + if (scm_is_number (m)) + m = scm_call_2 (proc, m, context ()->self_scm ()); + else + /* FIXME: constant error message. */ + warning (_ ("rehearsalMark must have integer value")); + } if (Text_interface::is_markup (m)) - text_->set_property ("text", m); + text_->set_property ("text", m); else - warning (_ ("mark label must be a markup object")); + warning (_ ("mark label must be a markup object")); } } -#include "translator.icc" +ADD_ACKNOWLEDGER (Mark_engraver, break_alignment); -ADD_ACKNOWLEDGER (Mark_engraver, bar_line); ADD_TRANSLATOR (Mark_engraver, - /* doc */ "This engraver will create RehearsalMark objects. " - "It puts them on top of all staves (which is taken from " - "the property @code{stavesFound}). If moving this 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 markFormatter stavesFound", - /* write */ ""); + /* doc */ + "Create @code{RehearsalMark} objects. It puts them on top of" + " all staves (which is taken from the property" + " @code{stavesFound}). If moving this engraver to a different" + " context, @ref{Staff_collecting_engraver} must move along," + " otherwise all marks end up on the same Y@tie{}location.", + + /* create */ + "RehearsalMark ", + + /* read */ + "markFormatter " + "rehearsalMark " + "stavesFound ", + + /* write */ + "" + );