]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/key-signature-interface.cc
(get_indexed_char): scale metrics by
[lilypond.git] / lily / key-signature-interface.cc
index 4666d4982a749dcdc6ee7fc537f1b13ebcc9c752..652eff7d168b64452cae3198446e2594ba1bba0f 100644 (file)
@@ -9,14 +9,10 @@
 */
 
 #include "item.hh"
-
-#include "stencil.hh"
-#include "paper-def.hh"
+#include "output-def.hh"
 #include "font-interface.hh"
 #include "staff-symbol-referencer.hh"
 #include "lookup.hh"
-#include "lily-guile.hh"
-#include "lily-proto.hh"
 #include "accidental-interface.hh"
 
 struct Key_signature_interface
@@ -30,8 +26,8 @@ struct Key_signature_interface
 /*
   FIXME: too much hardcoding here.
  */
-const int FLAT_TOP_PITCH=2; /* fes,ges,as and bes typeset in lower octave */
-const int SHARP_TOP_PITCH=4; /*  ais and bis typeset in lower octave */
+const int FLAT_TOP_PITCH =2; /* fes,ges,as and bes typeset in lower octave */
+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. 
@@ -46,9 +42,6 @@ const int NATURAL_TOP_PITCH = 4;
   the thinking to other parties.
 
   - TODO: put this in Scheme
-  
-  - lots of values trivially shared (key doesn't change very
-  often). Compute those once, and use that as cache for the rest.
 
   TODO: can  we do without c0pos? it's partly musical. 
 
@@ -56,10 +49,10 @@ const int NATURAL_TOP_PITCH = 4;
 int
 alteration_pos  (SCM what, int alter, int c0p)
 {
-  if (ly_c_pair_p (what))
-    return ly_scm2int (ly_car (what)) * 7 + ly_scm2int (ly_cdr (what)) + c0p;
+  if (scm_is_pair (what))
+    return scm_to_int (scm_car (what)) * 7 + scm_to_int (scm_cdr (what)) + c0p;
 
-  int p = ly_scm2int (what);
+  int p = scm_to_int (what);
 
   // Find the c in the range -4 through 2
   int from_bottom_pos = c0p + 4;
@@ -74,17 +67,18 @@ alteration_pos  (SCM what, int alter, int c0p)
     {
       p -= 7; /* Typeset below c_position */
     }
+  
   /* Provide for the four cases in which there's a glitch 
        it's a hack, but probably not worth  
        the effort of finding a nicer solution.
        --dl. */
-  if (c0==2 && alter >0 && p==3)
+  if (c0==2 && alter >0 && p ==3)
     p -= 7;
-  if (c0==-3 && alter>0 && p==-1)
+  if (c0==-3 && alter>0 && p ==-1)
     p += 7;
-  if (c0==-4 && alter<0 && p==-1)
+  if (c0==-4 && alter<0 && p ==-1)
     p += 7;
-  if (c0==-2 && alter<0 && p==-3)
+  if (c0==-2 && alter<0 && p ==-3)
     p += 7;
     
   return p + c0;
@@ -98,13 +92,13 @@ MAKE_SCHEME_CALLBACK (Key_signature_interface,print,1);
 SCM
 Key_signature_interface::print (SCM smob)
 {
-  Grob*me =unsmob_grob (smob);
+  Grob*me = unsmob_grob (smob);
 
   Real inter = Staff_symbol_referencer::staff_space (me)/2.0;
 
   SCM scm_style = me->get_property ("style");
   String style;
-  if (ly_c_symbol_p (scm_style))
+  if (scm_is_symbol (scm_style))
     {
       style = ly_symbol2string (scm_style);
     }
@@ -118,8 +112,8 @@ Key_signature_interface::print (SCM smob)
 
   SCM c0s = me->get_property ("c0-position");
   int c0p = 0;
-  if (ly_c_number_p (c0s))
-    c0p = ly_scm2int (c0s);
+  if (scm_is_number (c0s))
+    c0p = scm_to_int (c0s);
 
   /*
     SCM lists are stacks, so we work from right to left, ending with
@@ -127,12 +121,12 @@ Key_signature_interface::print (SCM smob)
   */
 
   Font_metric *fm = Font_interface::get_default_font (me);
-  for (SCM s = newas; ly_c_pair_p (s); s = ly_cdr (s))
+  for (SCM s = newas; scm_is_pair (s); s = scm_cdr (s))
     {
-      int alteration = ly_scm2int (ly_cdar (s));
+      int alteration = scm_to_int (scm_cdar (s));
       String font_char =
        Accidental_interface::get_fontcharname (style, alteration);
-      Stencil acc (fm->find_by_name ("accidentals-" + font_char));
+      Stencil acc (fm->find_by_name ("accidentals." + font_char));
 
       if (acc.is_empty ())
        {
@@ -140,7 +134,7 @@ Key_signature_interface::print (SCM smob)
        }
       else
        {
-         SCM what = ly_caar (s);
+         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);
@@ -152,28 +146,20 @@ Key_signature_interface::print (SCM smob)
     {
       SCM old = me->get_property ("old-accidentals");
       
-      /*
-       Add half a space between  cancellation and key sig.
-
-       As suggested by [Ross], p.148.
-       */
-      Interval x (0, inter);
-      Interval y (0,0);
-
-      mol.add_at_edge (X_AXIS, LEFT, Lookup::blank (Box (x,y)), 0, 0);
-
       Stencil natural;
-      if (ly_c_pair_p (old))
-       natural=Font_interface::get_default_font (me)->
+      if (scm_is_pair (old))
+       natural = Font_interface::get_default_font (me)->
            find_by_name (String ("accidentals-") + style + String ("0"));
       
-      for (; ly_c_pair_p (old); old = ly_cdr (old))
+
+      int last_pos = -100;
+      for (; scm_is_pair (old); old = scm_cdr (old))
         {
-         SCM found = scm_assoc (ly_caar (old), newas);
+         SCM found = scm_assoc (scm_caar (old), newas);
          if (found == SCM_BOOL_F
-             || ly_cdr (found) != ly_cdar (old))
+             || scm_cdr (found) != scm_cdar (old))
            {
-             SCM what = ly_caar (old);
+             SCM what = scm_caar (old);
              int alteration = 0;
              int pos = alteration_pos (what, alteration, c0p);
 
@@ -185,8 +171,13 @@ Key_signature_interface::print (SCM smob)
                has vertical edges on both sides. A little padding is
                needed to prevent collisions.
               */
-             Real padding = 0.1 ;
+             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;
             }
         }
     }
@@ -198,4 +189,4 @@ Key_signature_interface::print (SCM smob)
 
 ADD_INTERFACE (Key_signature_interface, "key-signature-interface",
   "A group of accidentals, to be printed as signature sign.",
-  "c0-position old-accidentals new-accidentals");
+  "style c0-position old-accidentals new-accidentals");