]> git.donarmstrong.com Git - lilypond.git/blobdiff - lily/vaticana-ligature-engraver.cc
Run grand-replace (issue 3765)
[lilypond.git] / lily / vaticana-ligature-engraver.cc
index 72fdb41f50317e2e0e21fca5cf1307e224bc8a6d..3749b5af882d2a6d8f2094f79fb80dc4a24a407e 100644 (file)
@@ -1,21 +1,38 @@
 /*
-  vaticana-ligature-engraver.cc -- implement Vaticana_ligature_engraver
+  This file is part of LilyPond, the GNU music typesetter.
 
-  source file of the GNU LilyPond music typesetter
+  Copyright (C) 2003--2014 Juergen Reuter <reuter@ipd.uka.de>
 
-  (c) 2003--2005 Juergen Reuter <reuter@ipd.uka.de>
+  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 "gregorian-ligature-engraver.hh"
+
+#include "font-interface.hh"
 #include "gregorian-ligature.hh"
-#include "vaticana-ligature.hh"
+#include "international.hh"
+#include "output-def.hh"
+#include "paper-column.hh"
+#include "separation-item.hh"
 #include "spanner.hh"
 #include "staff-symbol-referencer.hh"
-#include "font-interface.hh"
+#include "stream-event.hh"
+#include "vaticana-ligature.hh"
 #include "warn.hh"
-#include "output-def.hh"
-#include "paper-column.hh"
-
+#include "dot-column.hh"
+#include "rhythmic-head.hh"
+#include "pitch.hh"
 #include "translator.icc"
 
 /*
  * style ligatures for Gregorian chant notation.
  */
 
+/*
+ * TODO: Maybe move handling of dots/mora to
+ * Gregorian_ligature_engraver?  It's probably common for all types of
+ * Gregorian chant notation that have dotted notes.
+ *
+ * FIXME: The horizontal alignment of the mora column is bad (too far
+ * to the left), if the last dotted note is not the last primitive in
+ * the ligature.  Fortunately, in practice this bug should have no
+ * negative impact, since dotted notes appear within a ligature
+ * usually always at the end of the ligature, such that the bug never
+ * should apply for valid ligatures.
+ *
+ * TODO: Graduale Triplex, tempus per annum, hebdomada septima,
+ * alleluia (page 280) shows a counter-example for collecting dots
+ * always in a single column behind the ligature.  Maybe only the last
+ * two dots in a ligature should be collected and all other dots put
+ * behind or on top of the head?
+ */
 class Vaticana_ligature_engraver : public Gregorian_ligature_engraver
 {
 
 private:
   static bool
   need_extra_horizontal_space (int prev_prefix_set, int prefix_set,
-                              int context_info, int delta_pitch);
+                               int context_info, int delta_pitch);
   bool is_stacked_head (int prefix_set,
-                       int context_info);
-  Real align_heads (Array<Grob_info> primitives,
-                   Real flexa_width,
-                   Real thickness);
+                        int context_info);
+  Real align_heads (vector<Grob_info> const &primitives,
+                    Real flexa_width,
+                    Real thickness);
+  void check_for_prefix_loss (Item *primitive);
+  void check_for_ambiguous_dot_pitch (Grob_info primitive);
+  void add_mora_column (Paper_column *column);
+  vector<Grob_info> augmented_primitives_;
 
 public:
   TRANSLATOR_DECLARATIONS (Vaticana_ligature_engraver);
@@ -42,11 +81,30 @@ public:
 protected:
   virtual Spanner *create_ligature_spanner ();
   virtual void transform_heads (Spanner *ligature,
-                               Array<Grob_info> primitives);
+                                vector<Grob_info> const &primitives);
+  DECLARE_TRANSLATOR_LISTENER (pes_or_flexa);
+  DECLARE_TRANSLATOR_LISTENER (ligature);
 };
 
+IMPLEMENT_TRANSLATOR_LISTENER (Vaticana_ligature_engraver, pes_or_flexa);
+void
+Vaticana_ligature_engraver::listen_pes_or_flexa (Stream_event *ev)
+{
+  Gregorian_ligature_engraver::listen_pes_or_flexa (ev);
+}
+
+IMPLEMENT_TRANSLATOR_LISTENER (Vaticana_ligature_engraver, ligature);
+void
+Vaticana_ligature_engraver::listen_ligature (Stream_event *ev)
+{
+  Ligature_engraver::listen_ligature (ev);
+}
+
 Vaticana_ligature_engraver::Vaticana_ligature_engraver ()
 {
+  brew_ligature_primitive_proc
+    = Vaticana_ligature::brew_ligature_primitive_proc;
+  augmented_primitives_.clear ();
 }
 
 Spanner *
@@ -57,43 +115,43 @@ Vaticana_ligature_engraver::create_ligature_spanner ()
 
 bool
 Vaticana_ligature_engraver::is_stacked_head (int prefix_set,
-                                            int context_info)
+                                             int context_info)
 {
-  bool is_stacked_b;
+  bool is_stacked;
 
   // upper head of pes is stacked upon lower head of pes ...
-  is_stacked_b = context_info & PES_UPPER;
+  is_stacked = context_info & PES_UPPER;
 
   // ... unless this note starts a flexa
   if (context_info & FLEXA_LEFT)
-    is_stacked_b = false;
+    is_stacked = false;
 
   // ... or another pes
   if (context_info & PES_LOWER)
-    is_stacked_b = false;
+    is_stacked = false;
 
   // ... or the previous note is a semivocalis or inclinatum
   if (context_info & AFTER_DEMINUTUM)
-    is_stacked_b = false;
+    is_stacked = false;
 
   // auctum head is never stacked upon preceding note
   if (prefix_set & AUCTUM)
-    is_stacked_b = false;
+    is_stacked = false;
 
   // virga is never stacked upon preceding note
   if (prefix_set & VIRGA)
-    is_stacked_b = false;
+    is_stacked = false;
 
   // oriscus is never stacked upon preceding note
   if (prefix_set & ORISCUS)
-    is_stacked_b = false;
+    is_stacked = false;
 
   if ((prefix_set & DEMINUTUM)
       && ! (prefix_set & INCLINATUM)
       && (context_info & FLEXA_RIGHT))
-    is_stacked_b = true; // semivocalis head of deminutus form
+    is_stacked = true; // semivocalis head of deminutus form
 
-  return is_stacked_b;
+  return is_stacked;
 }
 
 /*
@@ -104,11 +162,11 @@ Vaticana_ligature_engraver::is_stacked_head (int prefix_set,
  */
 bool
 Vaticana_ligature_engraver::need_extra_horizontal_space (int prev_prefix_set, int prefix_set,
-                                                        int context_info, int delta_pitch)
+                                                         int context_info, int delta_pitch)
 {
   if (prev_prefix_set & VIRGA)
     /*
-     * After a virga, make a an additional small space such that the
+     * After a virga, make an additional small space such that the
      * appendix on the right side of the head does not touch the
      * following head.
      */
@@ -123,7 +181,7 @@ Vaticana_ligature_engraver::need_extra_horizontal_space (int prev_prefix_set, in
 
   if ((context_info & FLEXA_LEFT) && ! (context_info & PES_UPPER))
     /*
-     * Before a flexa (but not within a torculus), make a an
+     * Before a flexa (but not within a torculus), make an
      * additional small space such that the appendix on the left side
      * of the flexa does not touch the this head.
      */
@@ -141,14 +199,14 @@ Vaticana_ligature_engraver::need_extra_horizontal_space (int prev_prefix_set, in
 }
 
 Real
-Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
-                                        Real flexa_width,
-                                        Real thickness)
+Vaticana_ligature_engraver::align_heads (vector<Grob_info> const &primitives,
+                                         Real flexa_width,
+                                         Real thickness)
 {
   if (!primitives.size ())
     {
-      programming_error ("Vaticana_ligature: "
-                        "empty ligature [ignored]");
+      programming_error ("Vaticana_ligature:"
+                         " empty ligature [ignored]");
       return 0.0;
     }
 
@@ -159,7 +217,7 @@ Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
     = dynamic_cast<Item *> (primitives[0].grob ())->get_column ();
 
   Real join_thickness
-    = thickness * column->get_layout ()->get_dimension (ly_symbol2scm ("linethickness"));
+    = thickness * column->layout ()->get_dimension (ly_symbol2scm ("line-thickness"));
 
   /*
    * Amount of extra space two put between some particular
@@ -176,13 +234,13 @@ Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
 
   Item *prev_primitive = 0;
   int prev_prefix_set = 0;
-  for (int i = 0; i < primitives.size (); i++)
+  for (vsize i = 0; i < primitives.size (); i++)
     {
       Item *primitive = dynamic_cast<Item *> (primitives[i].grob ());
       int prefix_set
-       = scm_to_int (primitive->get_property ("prefix-set"));
+        = scm_to_int (primitive->get_property ("prefix-set"));
       int context_info
-       = scm_to_int (primitive->get_property ("context-info"));
+        = scm_to_int (primitive->get_property ("context-info"));
 
       /*
        * Get glyph_name, delta_pitch and context_info for this head.
@@ -190,30 +248,28 @@ Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
 
       SCM glyph_name_scm = primitive->get_property ("glyph-name");
       if (glyph_name_scm == SCM_EOL)
-       {
-         primitive->programming_error ("Vaticana_ligature:"
-                                       "undefined glyph-name -> "
-                                       "ignoring grob");
-         continue;
-       }
-      String glyph_name = ly_scm2string (glyph_name_scm);
+        {
+          primitive->programming_error ("Vaticana_ligature:"
+                                        " undefined glyph-name ->"
+                                        " ignoring grob");
+          continue;
+        }
+      string glyph_name = ly_scm2string (glyph_name_scm);
 
       int delta_pitch = 0;
       if (prev_primitive) /* urgh, need prev_primitive only here */
-       {
-         SCM delta_pitch_scm = prev_primitive->get_property ("delta-pitch");
-         if (delta_pitch_scm != SCM_EOL)
-           {
-             delta_pitch = scm_to_int (delta_pitch_scm);
-           }
-         else
-           {
-             primitive->programming_error ("Vaticana_ligature:"
-                                           "delta-pitch undefined -> "
-                                           "ignoring grob");
-             continue;
-           }
-       }
+        {
+          SCM delta_pitch_scm = prev_primitive->get_property ("delta-position");
+          if (delta_pitch_scm != SCM_EOL)
+            delta_pitch = scm_to_int (delta_pitch_scm);
+          else
+            {
+              primitive->programming_error ("Vaticana_ligature:"
+                                            " delta-position undefined ->"
+                                            " ignoring grob");
+              continue;
+            }
+        }
 
       /*
        * Now determine width and x-offset of head.
@@ -223,46 +279,45 @@ Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
       Real x_offset;
 
       if (context_info & STACKED_HEAD)
-       {
-         /*
-          * This head is stacked upon the previous one; hence, it
-          * does not contribute to the total width of the ligature,
-          * and its width is assumed to be 0.0.  Moreover, it is
-          * shifted to the left by its width such that the right side
-          * of this and the other head are horizontally aligned.
-          */
-         head_width = 0.0;
-         x_offset = join_thickness
-           - Font_interface::get_default_font (primitive)->
-           find_by_name ("noteheads." + glyph_name).extent (X_AXIS).length ();
-       }
-      else if (!String::compare (glyph_name, "flexa")
-              || !String::compare (glyph_name, ""))
-       {
-         /*
-          * This head represents either half of a flexa shape.
-          * Hence, it is assigned half the width of this shape.
-          */
-         head_width = 0.5 * flexa_width;
-         x_offset = 0.0;
-       }
+        {
+          /*
+           * This head is stacked upon the previous one; hence, it
+           * does not contribute to the total width of the ligature,
+           * and its width is assumed to be 0.0.  Moreover, it is
+           * shifted to the left by its width such that the right side
+           * of this and the other head are horizontally aligned.
+           */
+          head_width = 0.0;
+          x_offset = join_thickness
+                     - Font_interface::get_default_font (primitive)->
+                     find_by_name ("noteheads.s" + glyph_name).extent (X_AXIS).length ();
+        }
+      else if (glyph_name == "flexa" || glyph_name == "")
+        {
+          /*
+           * This head represents either half of a flexa shape.
+           * Hence, it is assigned half the width of this shape.
+           */
+          head_width = 0.5 * flexa_width;
+          x_offset = 0.0;
+        }
       else
-       {
-         /*
-          * This is a regular head, placed right to the previous one.
-          * Retrieve its width from corresponding font.
-          */
-         head_width
-           = Font_interface::get_default_font (primitive)->
-           find_by_name ("noteheads." + glyph_name).extent (X_AXIS).length ();
-         x_offset = 0.0;
-       }
+        {
+          /*
+           * This is a regular head, placed right to the previous one.
+           * Retrieve its width from corresponding font.
+           */
+          head_width
+            = Font_interface::get_default_font (primitive)->
+              find_by_name ("noteheads.s" + glyph_name).extent (X_AXIS).length ();
+          x_offset = 0.0;
+        }
 
       /*
        * Save the head's final x-offset.
        */
       primitive->set_property ("x-offset",
-                              scm_from_double (x_offset));
+                               scm_from_double (x_offset));
 
       /*
        * If the head is the 2nd head of a pes or flexa (but not a
@@ -270,45 +325,44 @@ Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
        * neighbour head (i.e. the previous head) by a vertical beam.
        */
       if ((context_info & PES_UPPER)
-         || ((context_info & FLEXA_RIGHT)
-             && ! (context_info & PES_LOWER)))
-       {
-         if (!prev_primitive)
-           {
-             primitive->programming_error ("vaticana ligature: add-join: "
-                                           "missing previous primitive");
-           }
-         else
-           {
-             prev_primitive->set_property ("add-join",
-                                           ly_bool2scm (true));
-
-             /*
-              * Create a small overlap of adjacent heads so that the join
-              * can be drawn perfectly between them.
-              */
-             ligature_width -= join_thickness;
-           }
-       }
-      else if (!String::compare (glyph_name, ""))
-       {
-         /*
-          * This is the 2nd (virtual) head of flexa shape.  Join it
-          * tightly with 1st head, i.e. do *not* add additional
-          * space, such that next head will not be off from the flexa
-          * shape.
-          */
-       }
+          || ((context_info & FLEXA_RIGHT)
+              && ! (context_info & PES_LOWER)))
+        {
+          if (!prev_primitive)
+            {
+              primitive->programming_error ("Vaticana ligature: add-join:"
+                                            " missing previous primitive");
+            }
+          else
+            {
+              prev_primitive->set_property ("add-join",
+                                            ly_bool2scm (true));
+
+              /*
+               * Create a small overlap of adjacent heads so that the join
+               * can be drawn perfectly between them.
+               */
+              ligature_width -= join_thickness;
+            }
+        }
+      else if (glyph_name == "")
+        {
+          /*
+           * This is the 2nd (virtual) head of flexa shape.  Join it
+           * tightly with 1st head, i.e. do *not* add additional
+           * space, such that next head will not be off from the flexa
+           * shape.
+           */
+        }
 
       if (need_extra_horizontal_space (prev_prefix_set, prefix_set,
-                                      context_info, delta_pitch))
-       ligature_width += extra_space;
+                                       context_info, delta_pitch))
+        ligature_width += extra_space;
 
       /*
        * Horizontally line-up this head to form a ligature.
        */
-      get_set_column (primitive, column);
-      primitive->translate_axis (ligature_width, X_AXIS);
+      move_related_items_to_column (primitive, column, ligature_width);
       ligature_width += head_width;
 
       prev_primitive = primitive;
@@ -336,22 +390,90 @@ Vaticana_ligature_engraver::align_heads (Array<Grob_info> primitives,
  * primitive were engraved as a stand-alone head.
  */
 void
-check_for_prefix_loss (Item *primitive)
+Vaticana_ligature_engraver::check_for_prefix_loss (Item *primitive)
 {
   int prefix_set
     = scm_to_int (primitive->get_property ("prefix-set"));
   if (prefix_set & ~PES_OR_FLEXA)
     {
-      String prefs = Gregorian_ligature::prefixes_to_str (primitive);
-      primitive->warning (_f ("ignored prefix (es) `%s' of this head according "
-                             "to restrictions of the selected ligature style",
-                             prefs.to_str0 ()));
+      string prefs = Gregorian_ligature::prefixes_to_str (primitive);
+      primitive->warning (_f ("ignored prefix(es) `%s' of this head"
+                              " according to restrictions of the selected"
+                              " ligature style",
+                              prefs.c_str ()));
+    }
+}
+
+void
+Vaticana_ligature_engraver::add_mora_column (Paper_column *column)
+{
+  if (augmented_primitives_.size () == 0) // no dot for column
+    return;
+  if (!column) // empty ligature???
+    {
+      augmented_primitives_[0].grob ()->
+      programming_error ("no paper column to add dot");
+      return;
+    }
+  Item *dotcol = make_item ("DotColumn", SCM_EOL);
+  dotcol->set_parent (column, X_AXIS);
+  for (vsize i = 0; i < augmented_primitives_.size (); i++)
+    {
+      Item *primitive
+        = dynamic_cast<Item *> (augmented_primitives_[i].grob ());
+      Item *dot = make_item ("Dots", primitive->self_scm ());
+      dot->set_property ("dot-count", scm_from_int (1));
+      dot->set_parent (primitive, Y_AXIS);
+      primitive->set_object ("dot", dot->self_scm ());
+      Dot_column::add_head (dotcol, primitive);
+
+      // FIXME: why isn't the dot picked up by Paper_column_engraver?
+      Separation_item::add_item (column, dot);
+    }
+}
+
+/*
+ * This function prints a warning, if the given primitive has the same
+ * pitch as at least one of the primitives already stored in the
+ * augmented_primitives_ array.
+ *
+ * The rationale of this check is, that, if there are two dotted
+ * primitives with the same pitch, then collecting all dots in a dot
+ * column behind the ligature leads to a notational ambiguity of to
+ * which head the corresponding dot refers.
+ *
+ * Such a case should be treated as a badly specified ligature.  The
+ * user should split the ligature to make the notation of dots
+ * unambiguous.
+ */
+void
+Vaticana_ligature_engraver::check_for_ambiguous_dot_pitch (Grob_info primitive)
+{
+  // TODO: Fix performance, which is currently O (n^2) (since this
+  // method is called O (n) times and takes O (n) steps in the for
+  // loop), but could be O (n) (by replacing the for loop by e.g. a
+  // bitmask based O (1) test); where n=<number of primitives in the
+  // ligature> (which is typically small (n<10), though).
+  Stream_event *new_cause = primitive.event_cause ();
+  int new_pitch = unsmob_pitch (new_cause->get_property ("pitch"))->steps ();
+  for (vsize i = 0; i < augmented_primitives_.size (); i++)
+    {
+      Stream_event *cause = augmented_primitives_[i].event_cause ();
+      int pitch = unsmob_pitch (cause->get_property ("pitch"))->steps ();
+      if (pitch == new_pitch)
+        {
+          primitive.grob ()->
+          warning (_ ("Ambiguous use of dots in ligature: there are"
+                      " multiple dotted notes with the same pitch."
+                      "  The ligature should be split."));
+          return; // supress multiple identical warnings
+        }
     }
 }
 
 void
 Vaticana_ligature_engraver::transform_heads (Spanner *ligature,
-                                            Array<Grob_info> primitives)
+                                             vector<Grob_info> const &primitives)
 {
   Real flexa_width = robust_scm2double (ligature->get_property ("flexa-width"), 2);
 
@@ -361,36 +483,56 @@ Vaticana_ligature_engraver::transform_heads (Spanner *ligature,
   int prev_prefix_set = 0;
   int prev_context_info = 0;
   int prev_delta_pitch = 0;
-  String prev_glyph_name = "";
-  for (int i = 0; i < primitives.size (); i++)
+  string prev_glyph_name = "";
+  augmented_primitives_.clear ();
+  for (vsize i = 0; i < primitives.size (); i++)
     {
       Item *primitive = dynamic_cast<Item *> (primitives[i].grob ());
 
       int delta_pitch;
-      SCM delta_pitch_scm = primitive->get_property ("delta-pitch");
+      SCM delta_pitch_scm = primitive->get_property ("delta-position");
       if (delta_pitch_scm != SCM_EOL)
-       {
-         delta_pitch = scm_to_int (delta_pitch_scm);
-       }
+        delta_pitch = scm_to_int (delta_pitch_scm);
       else
-       {
-         primitive->programming_error ("Vaticana_ligature:"
-                                       "delta-pitch undefined -> "
-                                       "ignoring grob");
-         continue;
-       }
+        {
+          primitive->programming_error ("Vaticana_ligature:"
+                                        " delta-position undefined ->"
+                                        " ignoring grob");
+          continue;
+        }
 
       /* retrieve & complete prefix_set and context_info */
       int prefix_set
-       = scm_to_int (primitive->get_property ("prefix-set"));
+        = scm_to_int (primitive->get_property ("prefix-set"));
       int context_info
-       = scm_to_int (primitive->get_property ("context-info"));
+        = scm_to_int (primitive->get_property ("context-info"));
+
+      if (Rhythmic_head::dot_count (primitive) > 0)
+        // remove dots from primitive and add remember primitive for
+        // creating a dot column
+        {
+          Rhythmic_head::get_dots (primitive)->set_property ("dot-count",
+                                                             scm_from_int (0));
+          // TODO: Maybe completely remove grob "Dots" (dots->suicide
+          // () ?) rather than setting property "dot-count" to 0.
+
+          check_for_ambiguous_dot_pitch (primitives[i]);
+          augmented_primitives_.push_back (primitives[i]);
+        }
+      else if (augmented_primitives_.size () > 0)
+        {
+          primitive->warning (_ ("This ligature has a dotted head followed by"
+                                 " a non-dotted head.  The ligature should be"
+                                 " split after the last dotted head before"
+                                 " this head."));
+        }
+
       if (is_stacked_head (prefix_set, context_info))
-       {
-         context_info |= STACKED_HEAD;
-         primitive->set_property ("context-info",
-                                  scm_from_int (context_info));
-       }
+        {
+          context_info |= STACKED_HEAD;
+          primitive->set_property ("context-info",
+                                   scm_from_int (context_info));
+        }
 
       /*
        * Now determine which head to typeset (this is context sensitive
@@ -398,122 +540,120 @@ Vaticana_ligature_engraver::transform_heads (Spanner *ligature,
        * this decision must be made here in the engraver rather than in
        * the backend).
        */
-      String glyph_name;
+      string glyph_name;
       if (prefix_set & VIRGA)
-       {
-         glyph_name = "svaticana.punctum";
-         primitive->set_property ("add-stem", ly_bool2scm (true));
-       }
+        {
+          glyph_name = "vaticana.punctum";
+          primitive->set_property ("add-stem", ly_bool2scm (true));
+        }
       else if (prefix_set & QUILISMA)
-       glyph_name = "svaticana.quilisma";
+        glyph_name = "vaticana.quilisma";
       else if (prefix_set & ORISCUS)
-       glyph_name = "ssolesmes.oriscus";
+        glyph_name = "solesmes.oriscus";
       else if (prefix_set & STROPHA)
-       if (prefix_set & AUCTUM)
-         glyph_name = "ssolesmes.stropha.aucta";
-       else glyph_name = "ssolesmes.stropha";
+        if (prefix_set & AUCTUM)
+          glyph_name = "solesmes.stropha.aucta";
+        else glyph_name = "solesmes.stropha";
       else if (prefix_set & INCLINATUM)
-       if (prefix_set & AUCTUM)
-         glyph_name = "ssolesmes.incl.auctum";
-       else if (prefix_set & DEMINUTUM)
-         glyph_name = "ssolesmes.incl.parvum";
-       else
-         glyph_name = "svaticana.inclinatum";
+        if (prefix_set & AUCTUM)
+          glyph_name = "solesmes.incl.auctum";
+        else if (prefix_set & DEMINUTUM)
+          glyph_name = "solesmes.incl.parvum";
+        else
+          glyph_name = "vaticana.inclinatum";
       else if (prefix_set & DEMINUTUM)
-       if (i == 0)
-         {
-           // initio debilis
-           glyph_name = "svaticana.reverse.plica";
-         }
-       else if (prev_delta_pitch > 0)
-         {
-           // epiphonus
-           if (! (prev_context_info & FLEXA_RIGHT))
-             /* correct head of previous primitive */
-             if (prev_delta_pitch > 1)
-               {
-                 prev_glyph_name = "svaticana.epiphonus";
-                 glyph_name = "svaticana.plica";
-               }
-             else
-               {
-                 prev_glyph_name = "svaticana.vepiphonus";
-                 glyph_name = "svaticana.vplica";
-               }
-         }
-       else if (prev_delta_pitch < 0)
-         {
-           // cephalicus
-           if (! (prev_context_info & FLEXA_RIGHT))
-             /* correct head of previous primitive */
-             {
-               if (i > 1)
-                 {
-                   /* cephalicus head with fixed size cauda */
-                   prev_glyph_name = "svaticana.inner.cephalicus";
-                 }
-               else
-                 {
-                   /* cephalicus head without cauda */
-                   prev_glyph_name = "svaticana.cephalicus";
-                 }
-
-               /*
-                * Flexa has no variable size cauda if its left head is
-                * stacked on the right head.  This is true for
-                * cephalicus.  Hence, remove the cauda.
-                *
-                * Urgh: for the current implementation, this rule only
-                * applies for cephalicus; but it is a fundamental rule.
-                * Therefore, the following line of code should be
-                * placed somewhere else.
-                */
-               prev_primitive->set_property ("add-cauda",
-                                             ly_bool2scm (false));
-             }
-             if (prev_delta_pitch < - 1)
-               {
-                 glyph_name = "svaticana.reverse.plica";
-               }
-             else
-               {
-                 glyph_name = "svaticana.reverse.vplica";
-               }
-         }
-       else // (prev_delta_pitch == 0)
-         {
-           primitive->programming_error ("Vaticana_ligature:"
-                                         "deminutum head must have different "
-                                         "pitch -> ignoring grob");
-         }
+        {
+          if (i == 0)
+            {
+              // initio debilis
+              glyph_name = "vaticana.reverse.plica";
+            }
+          else if (prev_delta_pitch > 0)
+            {
+              // epiphonus
+              if (! (prev_context_info & FLEXA_RIGHT))
+                {
+                  /* correct head of previous primitive */
+                  if (prev_delta_pitch > 1)
+                    prev_glyph_name = "vaticana.epiphonus";
+                  else
+                    prev_glyph_name = "vaticana.vepiphonus";
+                }
+              if (prev_delta_pitch > 1)
+                glyph_name = "vaticana.plica";
+              else
+                glyph_name = "vaticana.vplica";
+            }
+          else if (prev_delta_pitch < 0)
+            {
+              // cephalicus
+              if (! (prev_context_info & FLEXA_RIGHT))
+                /* correct head of previous primitive */
+                {
+                  if (i > 1)
+                    {
+                      /* cephalicus head with fixed size cauda */
+                      prev_glyph_name = "vaticana.inner.cephalicus";
+                    }
+                  else
+                    {
+                      /* cephalicus head without cauda */
+                      prev_glyph_name = "vaticana.cephalicus";
+                    }
+
+                  /*
+                   * Flexa has no variable size cauda if its left head is
+                   * stacked on the right head.  This is true for
+                   * cephalicus.  Hence, remove the cauda.
+                   *
+                   * Urgh: for the current implementation, this rule only
+                   * applies for cephalicus; but it is a fundamental rule.
+                   * Therefore, the following line of code should be
+                   * placed somewhere else.
+                   */
+                  prev_primitive->set_property ("add-cauda",
+                                                ly_bool2scm (false));
+                }
+              if (prev_delta_pitch < - 1)
+                glyph_name = "vaticana.reverse.plica";
+              else
+                glyph_name = "vaticana.reverse.vplica";
+            }
+          else // (prev_delta_pitch == 0)
+            {
+              primitive->programming_error ("Vaticana_ligature:"
+                                            " deminutum head must have different"
+                                            " pitch -> ignoring grob");
+            }
+        }
       else if (prefix_set & (CAVUM | LINEA))
-       if ((prefix_set & CAVUM) && (prefix_set & LINEA))
-         glyph_name = "svaticana.linea.punctum.cavum";
-       else if (prefix_set & CAVUM)
-         glyph_name = "svaticana.punctum.cavum";
-       else
-         glyph_name = "svaticana.linea.punctum";
+        if ((prefix_set & CAVUM) && (prefix_set & LINEA))
+          glyph_name = "vaticana.linea.punctum.cavum";
+        else if (prefix_set & CAVUM)
+          glyph_name = "vaticana.punctum.cavum";
+        else
+          glyph_name = "vaticana.linea.punctum";
       else if (prefix_set & AUCTUM)
-       if (prefix_set & ASCENDENS)
-         glyph_name = "ssolesmes.auct.asc";
-       else
-         glyph_name = "ssolesmes.auct.desc";
+        if (prefix_set & ASCENDENS)
+          glyph_name = "solesmes.auct.asc";
+        else
+          glyph_name = "solesmes.auct.desc";
       else if ((context_info & STACKED_HEAD)
-              && (context_info & PES_UPPER))
-       if (prev_delta_pitch > 1)
-         glyph_name = "svaticana.upes";
-       else
-         glyph_name = "svaticana.vupes";
+               && (context_info & PES_UPPER))
+        if (prev_delta_pitch > 1)
+          glyph_name = "vaticana.upes";
+        else
+          glyph_name = "vaticana.vupes";
       else
-       glyph_name = "svaticana.punctum";
+        glyph_name = "vaticana.punctum";
 
       /*
        * This head needs a cauda, if it starts a flexa, is not the upper
        * head of a pes, and if it is a punctum.
        */
       if ((context_info & FLEXA_LEFT) && ! (context_info & PES_UPPER))
-       if (!String::compare (glyph_name, "svaticana.punctum"))
-         primitive->set_property ("add-cauda", ly_bool2scm (true));
+        if (glyph_name == "vaticana.punctum")
+          primitive->set_property ("add-cauda", ly_bool2scm (true));
 
       /*
        * Execptional rule for porrectus:
@@ -523,21 +663,21 @@ Vaticana_ligature_engraver::transform_heads (Spanner *ligature,
        * the previous head into a single curved flexa shape.
        */
       if ((context_info & FLEXA_RIGHT) && (context_info & PES_LOWER))
-       {
-         check_for_prefix_loss (prev_primitive);
-         prev_glyph_name = "flexa";
-         prev_primitive->set_property ("flexa-height",
-                                       scm_from_int (prev_delta_pitch));
-         prev_primitive->set_property ("flexa-width",
-                                       scm_from_double (flexa_width));
-         bool add_cauda = !(prev_prefix_set && PES_OR_FLEXA);
-         prev_primitive->set_property ("add-cauda",
-                                       ly_bool2scm (add_cauda));
-         check_for_prefix_loss (primitive);
-         glyph_name = "";
-         primitive->set_property ("flexa-width",
-                                  scm_from_double (flexa_width));
-       }
+        {
+          check_for_prefix_loss (prev_primitive);
+          prev_glyph_name = "flexa";
+          prev_primitive->set_property ("flexa-height",
+                                        scm_from_int (prev_delta_pitch));
+          prev_primitive->set_property ("flexa-width",
+                                        scm_from_double (flexa_width));
+          bool add_cauda = !(prev_prefix_set && PES_OR_FLEXA);
+          prev_primitive->set_property ("add-cauda",
+                                        ly_bool2scm (add_cauda));
+          check_for_prefix_loss (primitive);
+          glyph_name = "";
+          primitive->set_property ("flexa-width",
+                                   scm_from_double (flexa_width));
+        }
 
       /*
        * Exceptional rule for pes:
@@ -548,20 +688,22 @@ Vaticana_ligature_engraver::transform_heads (Spanner *ligature,
        * current head.
        */
       if (prefix_set & PES_OR_FLEXA)
-       {
-         if ((context_info & PES_UPPER) && (context_info & STACKED_HEAD))
-           {
-             if (!String::compare (prev_glyph_name, "svaticana.punctum"))
-               if (prev_delta_pitch > 1)
-                 prev_glyph_name = "svaticana.lpes";
-               else
-                 prev_glyph_name = "svaticana.vlpes";
-           }
-       }
+        {
+          if ((context_info & PES_UPPER) && (context_info & STACKED_HEAD))
+            {
+              if (prev_glyph_name == "vaticana.punctum")
+                {
+                  if (prev_delta_pitch > 1)
+                    prev_glyph_name = "vaticana.lpes";
+                  else
+                    prev_glyph_name = "vaticana.vlpes";
+                }
+            }
+        }
 
       if (prev_primitive)
-       prev_primitive->set_property ("glyph-name",
-                                     scm_makfrom0str (prev_glyph_name.to_str0 ()));
+        prev_primitive->set_property ("glyph-name",
+                                      ly_string2scm (prev_glyph_name));
 
       /*
        * In the backend, flexa shapes and joins need to know about line
@@ -579,28 +721,40 @@ Vaticana_ligature_engraver::transform_heads (Spanner *ligature,
     }
 
   prev_primitive->set_property ("glyph-name",
-                               scm_makfrom0str (prev_glyph_name.to_str0 ()));
+                                ly_string2scm (prev_glyph_name));
 
   align_heads (primitives, flexa_width, thickness);
 
+  // append all dots to paper column of ligature's last head
+  add_mora_column (prev_primitive->get_column ());
+
 #if 0 // experimental code to collapse spacing after ligature
   /* TODO: set to max (old/new spacing-increment), since other
      voices/staves also may want to set this property. */
   Item *first_primitive = dynamic_cast<Item *> (primitives[0].grob ());
   Paper_column *paper_column = first_primitive->get_column ();
-  paper_column->warning (_f ("Vaticana_ligature_engraver: "
-                            "setting `spacing-increment = %f': ptr =%ul",
-                            ligature_width, paper_column));
+  paper_column->warning (_f ("Vaticana_ligature_engraver:"
+                             " setting `spacing-increment = %f': ptr =%ul",
+                             ligature_width, paper_column));
   paper_column->
-    set_property ("forced-spacing", scm_from_double (ligature_width));
+  set_property ("forced-spacing", scm_from_double (ligature_width));
 #endif
 }
 
-ADD_ACKNOWLEDGER(Vaticana_ligature_engraver, rest);
-ADD_ACKNOWLEDGER(Vaticana_ligature_engraver, note_head);
+ADD_ACKNOWLEDGER (Vaticana_ligature_engraver, rest);
+ADD_ACKNOWLEDGER (Vaticana_ligature_engraver, ligature_head);
 ADD_TRANSLATOR (Vaticana_ligature_engraver,
-               /* descr */ "Handles ligatures by glueing special ligature heads together.",
-               /* creats*/ "VaticanaLigature",
-               /* accepts */ "ligature-event",
-               /* reads */ "",
-               /* write */ "");
+                /* doc */
+                "Handle ligatures by glueing special ligature heads"
+                " together.",
+
+                /* create */
+                "VaticanaLigature "
+                "DotColumn ",
+
+                /* read */
+                "",
+
+                /* write */
+                ""
+               );