removed comment

[lilypond.git] / lily / key-signature-interface.cc

diff --git a/lily/key-signature-interface.cc b/lily/key-signature-interface.cc
index 375bac59b323a982f55c12b79d8a61395e28c285..8b45adb2c85fea569f963c5a44ee83a4e3a53ed6 100644 (file)
--- a/lily/key-signature-interface.cc
+++ b/lily/key-signature-interface.cc
@@ -3,23 +3,24 @@
 
   source file of the GNU LilyPond music typesetter
 
-  (c) 1996--2005 Han-Wen Nienhuys <hanwen@cs.uu.nl>
+  (c) 1996--2006 Han-Wen Nienhuys <hanwen@xs4all.nl>
 
   keyplacement by Mats Bengtsson
 */
 
+#include "accidental-interface.hh"
+#include "font-interface.hh"
+#include "international.hh"
 #include "item.hh"
+#include "lookup.hh"
 #include "output-def.hh"
-#include "font-interface.hh"
 #include "staff-symbol-referencer.hh"
-#include "lookup.hh"
-#include "accidental-interface.hh"
 
 struct Key_signature_interface
 {
   DECLARE_SCHEME_CALLBACK (print, (SCM));
 
-  static bool has_interface (Grob *);
+  DECLARE_GROB_INTERFACE();
 };
 
 /*
@@ -30,6 +31,9 @@ const int SHARP_TOP_PITCH = 4; /*  ais and bis typeset in lower octave */
 
 /*
   TODO: look this up. I'm not sure where the naturals ought to go.
+
+  COMMENT: Current implementation does not use the NATURAL_TOP_PITCH for anything,
+           always typesets naturals in the same place as the thing they cancel. -rz
 */
 const int NATURAL_TOP_PITCH = 4;
 
@@ -51,8 +55,8 @@ alteration_pos (SCM what, int alter, int c0p)
 
   // Find the c in the range -4 through 2
   int from_bottom_pos = c0p + 4;
-  from_bottom_pos = from_bottom_pos%7;
-  from_bottom_pos = (from_bottom_pos + 7)%7; // Precaution to get positive.
+  from_bottom_pos = from_bottom_pos % 7;
+  from_bottom_pos = (from_bottom_pos + 7) % 7; // Precaution to get positive.
   int c0 = from_bottom_pos - 4;
 
   if ((alter < 0 && ((p > FLAT_TOP_PITCH) || (p + c0 > 4)) && (p + c0 > 1))
@@ -86,22 +90,18 @@ MAKE_SCHEME_CALLBACK (Key_signature_interface, print, 1);
 SCM
 Key_signature_interface::print (SCM smob)
 {
-  Grob *me = unsmob_grob (smob);
+  Item *me = dynamic_cast<Item *> (unsmob_grob (smob));
 
   Real inter = Staff_symbol_referencer::staff_space (me) / 2.0;
 
   SCM scm_style = me->get_property ("style");
-  String style;
+  string style;
   if (scm_is_symbol (scm_style))
-    {
-      style = ly_symbol2string (scm_style);
-    }
+    style = ly_symbol2string (scm_style);
   else
-    {
-      style = "";
-    }
+    style = "";
 
-  SCM newas = me->get_property ("new-accidentals");
+  SCM newas = me->get_property ("alteration-alist");
   Stencil mol;
 
   SCM c0s = me->get_property ("c0-position");
@@ -109,69 +109,47 @@ Key_signature_interface::print (SCM smob)
   if (scm_is_number (c0s))
     c0p = scm_to_int (c0s);
 
+  bool is_cancellation = me->internal_has_interface
+    (ly_symbol2scm ("key-cancellation-interface"));
+
   /*
     SCM lists are stacks, so we work from right to left, ending with
     the cancellation signature.
   */
 
+  int last_pos = -1000;
   Font_metric *fm = Font_interface::get_default_font (me);
   for (SCM s = newas; scm_is_pair (s); s = scm_cdr (s))
     {
       int alteration = scm_to_int (scm_cdar (s));
-      String font_char
-       = Accidental_interface::get_fontcharname (style, alteration);
+      string font_char
+       = Accidental_interface::get_fontcharname (style,
+                                                 is_cancellation
+                                                 ? 0
+                                                 : alteration);
       Stencil acc (fm->find_by_name ("accidentals." + font_char));
 
       if (acc.is_empty ())
-       {
-         me->warning (_f ("accidental `%s' not found", font_char));
-       }
+       me->warning (_f ("accidental `%s' not found", font_char));
       else
        {
          SCM what = scm_caar (s);
          int pos = alteration_pos (what, alteration, c0p);
          acc.translate_axis (pos * inter, Y_AXIS);
-         mol.add_at_edge (X_AXIS, LEFT, acc, 0, 0);
-       }
-    }
-
-  Item *it = dynamic_cast<Item *> (me);
-  if (it->break_status_dir () != RIGHT)
-    {
-      SCM old = me->get_property ("old-accidentals");
 
-      Stencil natural;
-      if (scm_is_pair (old))
-       natural = Font_interface::get_default_font (me)->
-         find_by_name (String ("accidentals.") + style + String ("0"));
-
-      int last_pos = -100;
-      for (; scm_is_pair (old); old = scm_cdr (old))
-       {
-         SCM found = scm_assoc (scm_caar (old), newas);
-         if (found == SCM_BOOL_F
-             || scm_cdr (found) != scm_cdar (old))
-           {
-             SCM what = scm_caar (old);
-             int alteration = 0;
-             int pos = alteration_pos (what, alteration, c0p);
-
-             Stencil m = natural;
-             m.translate_axis (pos * inter, Y_AXIS);
-
-             /*
-               The natural sign (unlike flat & sharp)
-               has vertical edges on both sides. A little padding is
-               needed to prevent collisions.
-             */
-             Real padding = 0.0;
-             if (last_pos < pos + 2
-                 && last_pos > pos - 6)
-               padding = 0.3;
-
-             mol.add_at_edge (X_AXIS, LEFT, m, padding, 0);
-             last_pos = pos;
-           }
+         /*
+           The natural sign (unlike flat & sharp)
+           has vertical edges on both sides. A little padding is
+           needed to prevent collisions.
+         */
+         Real padding = 0.0;
+         if (is_cancellation
+             && last_pos < pos + 2
+             && last_pos > pos - 6)
+           padding = 0.3;
+
+         mol.add_at_edge (X_AXIS, LEFT, acc, padding, 0);
+         last_pos = pos;
        }
     }
 
@@ -180,6 +158,6 @@ Key_signature_interface::print (SCM smob)
   return mol.smobbed_copy ();
 }
 
-ADD_INTERFACE (Key_signature_interface, "key-signature-interface",
+ADD_INTERFACE (Key_signature_interface,
               "A group of accidentals, to be printed as signature sign.",
-              "style c0-position old-accidentals new-accidentals");
+              "style c0-position alteration-alist");