X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=lily%2Faccidental.cc;h=1bbf7e8a058ba8092ffb13925aefbb71608a0348;hb=7eb311a6e3235709ed969208210a6b49f129d639;hp=571c33cbea497e5c8db3cf9eaa305c5b154dc952;hpb=ade2bb7da653fbd9d670590afd67376fe69c59dd;p=lilypond.git

diff --git a/lily/accidental.cc b/lily/accidental.cc
index 571c33cbea..1bbf7e8a05 100644
--- a/lily/accidental.cc
+++ b/lily/accidental.cc
@@ -1,9 +1,20 @@
 /*
-  accidental.cc -- implement Accidental_interface
+  This file is part of LilyPond, the GNU music typesetter.
 
-  source file of the GNU LilyPond music typesetter
+  Copyright (C) 2001--2010 Han-Wen Nienhuys <hanwen@xs4all.nl>
 
-  (c) 2001--2007 Han-Wen Nienhuys <hanwen@xs4all.nl>
+  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 <http://www.gnu.org/licenses/>.
 */
 
 #include "accidental-interface.hh"
@@ -31,14 +42,30 @@ parenthesize (Grob *me, Stencil m)
   return m;
 }
 
+/* If this gets called before line breaking, we will return a non-trivial
+   extent even if we belong to a tie and won't actually get printed. */
+static SCM
+get_extent (Grob *me, Axis a)
+{
+  Stencil *s = unsmob_stencil (Accidental_interface::get_stencil (me));
+
+  if (s)
+    return ly_interval2scm (s->extent (a));
+  return ly_interval2scm (Interval ());
+}
 
-/* This callback exists for the sole purpose of allowing us to override
-   its pure equivalent to accidental-interface::pure-height */
 MAKE_SCHEME_CALLBACK (Accidental_interface, height, 1);
 SCM
 Accidental_interface::height (SCM smob)
 {
-  return Grob::stencil_height (smob);
+  return get_extent (unsmob_grob (smob), Y_AXIS);
+}
+
+MAKE_SCHEME_CALLBACK (Accidental_interface, width, 1);
+SCM
+Accidental_interface::width (SCM smob)
+{
+  return get_extent (unsmob_grob (smob), X_AXIS);
 }
 
 MAKE_SCHEME_CALLBACK (Accidental_interface, pure_height, 3);
@@ -49,11 +76,17 @@ Accidental_interface::pure_height (SCM smob, SCM start_scm, SCM)
   int start = scm_to_int (start_scm);
   int rank = me->get_column ()->get_rank ();
 
-  bool visible = to_boolean (me->get_property ("forced"))
-    || !unsmob_grob (me->get_object ("tie"))
-    || rank != start + 1; /* we are in the middle of a line */
+  if (to_boolean (me->get_property ("forced"))
+      || !unsmob_grob (me->get_object ("tie"))
+      || (rank == start + 1 && /* we are at the start of a line */
+	  !to_boolean (me->get_property ("hide-tied-accidental-after-break"))))
+    {
+      Stencil *s = unsmob_stencil (get_stencil (me));
+      if (s)
+	return ly_interval2scm (s->extent (Y_AXIS));
+    }
 
-  return visible ? Grob::stencil_height (smob) : ly_interval2scm (Interval ());
+  return ly_interval2scm (Interval ());
 }
 
 vector<Box>
@@ -72,9 +105,13 @@ Accidental_interface::accurate_boxes (Grob *me, Grob **common)
   if (!to_boolean (me->get_property ("restore-first"))
       && !parens)
     {
-      Rational alteration
-	= robust_scm2rational (me->get_property ("alteration"), 0);
-      if (alteration == FLAT_ALTERATION)
+      SCM alist = me->get_property ("glyph-name-alist");
+      SCM alt = me->get_property ("alteration");
+      string glyph_name = robust_scm2string (ly_assoc_get (alt, alist, SCM_BOOL_F),
+					     "");
+      
+      if (glyph_name == "accidentals.flat"
+	  || glyph_name == "accidentals.mirroredflat")
 	{
 	  Box stem = b;
 	  Box bulb = b;
@@ -83,7 +120,9 @@ Accidental_interface::accurate_boxes (Grob *me, Grob **common)
 	    we could make the stem thinner, but that places the flats
 	    really close.
 	  */
-	  stem[X_AXIS][RIGHT] *= .5;
+	  Direction bulb_dir =
+	    glyph_name == "accidentals.mirroredflat" ? LEFT : RIGHT;
+	  stem[X_AXIS][bulb_dir] = stem[X_AXIS].center ();
 
 	  /*
 	    To prevent vertical alignment for 6ths
@@ -94,7 +133,7 @@ Accidental_interface::accurate_boxes (Grob *me, Grob **common)
 	  boxes.push_back (bulb);
 	  boxes.push_back (stem);
 	}
-      else if (alteration == NATURAL_ALTERATION)
+      else if (glyph_name ==  "accidentals.natural")
 	{
 	  Box lstem = b;
 	  Box rstem = b;
@@ -136,13 +175,20 @@ Accidental_interface::print (SCM smob)
   Grob *me = unsmob_grob (smob);
   Grob *tie = unsmob_grob (me->get_object ("tie"));
 
-  if (tie && !tie->original ()
-      && !to_boolean (me->get_property ("forced")))
+  if (tie &&
+      (to_boolean (me->get_property ("hide-tied-accidental-after-break"))
+       || (!tie->original () && !to_boolean (me->get_property ("forced")))))
     {
       me->suicide ();
       return SCM_EOL;
     }
-  
+
+  return get_stencil (me);
+}
+
+SCM
+Accidental_interface::get_stencil (Grob *me)
+{
   Font_metric *fm = Font_interface::get_default_font (me);
 
   SCM alist = me->get_property ("glyph-name-alist");
@@ -156,7 +202,7 @@ Accidental_interface::print (SCM smob)
       return SCM_EOL;
     }
   
-  Stencil mol (fm->find_by_name (scm_i_string_chars (glyph_name)));
+  Stencil mol (fm->find_by_name (ly_scm2string (glyph_name)));
   if (to_boolean (me->get_property ("restore-first")))
     {
       /*
@@ -179,14 +225,15 @@ Accidental_interface::print (SCM smob)
 
   
 ADD_INTERFACE (Accidental_interface,
-	       "a single accidental",
+	       "A single accidental.",
 	       
-	       /* props */
+	       /* properties */
 	       "alteration "
 	       "avoid-slur "
 	       "forced "
+	       "glyph-name-alist "
+	       "hide-tied-accidental-after-break "
 	       "parenthesized "
 	       "restore-first "
-	       "glyph-name-alist "
 	       "tie "
 	       );