[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: \incipit sometimes returns programming error

From: Jean Abou Samra
Subject: Re: \incipit sometimes returns programming error
Date: Thu, 26 May 2022 23:33:29 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1

Le 26/05/2022 à 18:47, Thomas Morley a écrit :
Am Do., 26. Mai 2022 um 17:36 Uhr schrieb Jean Abou Samra <jean@abou-samra.fr>:

Le 26/05/2022 à 17:12, Thomas Morley a écrit :


\score {
    \new Staff \with { instrumentName = "" }  { \incipit { c'1. } R1 }
    \layout {
      indent = 5\cm
      incipit-width = 3\cm

with recent versions it returns:
programming error: Loose column does not have right side to attach to.

First bad comit is:
commit 8ae26d8330c603d249fec5953a887de9fbcbe31c
Author: Dan Eble <nine.fierce.ballads@gmail.com>
Date:   Thu Mar 10 20:08:31 2022 -0500

      Refactor BarLine engraving


The example above can be fixed by inserting an explicit \bar "" at end
of incipit-music.


That looks like an old bug uncovered to me. \incipit
uses a MensuralStaff, and indeed I can reproduce it

\version "2.23.10"

\new MensuralStaff {

The definition of MensuralStaff contains measureBarType = "",
and this also triggers the bug:

\version "2.23.10"

\new Staff \with {
    measureBarType = ""
} {

which is an instance of our old


I can't tell why the commit "Refactor BarLine engraving" had that
effect (and I'm not investigating). Dan, any thoughts? Should
MensuralStaff simply be changed to have measureBarType = ##f
(instead of "")?


Currently I'm testing LSR-upgrade to current master, anticipating the
next stable. In the past doing so served me well as soon as the next
stable is actually out and updating LSR is really a TODO-NOW.
I stumbled upon a handful snippets (I may post about them, if I don't
find my way), one of them is incipit.ly,
It works fine with 2.22. and before, I wonder why #4084 raises its
head now and not before.

Wouldn't it be a possibility to just change  our `incipit':
$ git diff
diff --git a/ly/property-init.ly b/ly/property-init.ly
index f8e6ee37be..c0605757be 100644
--- a/ly/property-init.ly
+++ b/ly/property-init.ly
@@ -323,6 +323,7 @@ incipit =
                  \override InstrumentName.self-alignment-X = #align-x
                  \override InstrumentName.self-alignment-Y = #align-y
                  instrumentName = #instrument-name
+                measureBarType = ##f

At least it fixes the here reported bug.

It doesn't fix \new MensuralStaff { c'1. } though.
I'm for understanding what happened before changing


reply via email to

[Prev in Thread] Current Thread [Next in Thread]