[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Align metronome mark at time signature or first musical element. Fix
From: |
Han-Wen Nienhuys |
Subject: |
Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041) |
Date: |
Sun, 29 Aug 2010 17:46:07 -0300 |
On Sun, Aug 29, 2010 at 5:21 PM, Jan Nieuwenhuizen
<address@hidden> wrote:
> It's not cyclic, the idea was about not crashing when the data
> structure is not a list at all.
>
> I think Neil had a remark about this.
Most of the similar code uses scm_c_memq instead, which skips the
checking. On closer inspection, scm_ilength (which does the actual
checking) also catches other errors besides circular lists.
We could make more strict checking a habit, but it's not really a
scalable, since many routines require more complex data structures as
input parameters.
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), jan . nieuwenhuizen, 2010/08/24
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), n . puttock, 2010/08/25
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), n . puttock, 2010/08/26
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), jan . nieuwenhuizen, 2010/08/27
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), hanwenn, 2010/08/29
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), reinhold . kainhofer, 2010/08/29
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), hanwenn, 2010/08/29
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), n . puttock, 2010/08/29
- Re: Align metronome mark at time signature or first musical element. Fixes #684. (issue1579041), n . puttock, 2010/08/29