[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Re: Duration dots and Bar lines on custom Staves.
From: |
Han-Wen Nienhuys |
Subject: |
Re: [PATCH] Re: Duration dots and Bar lines on custom Staves. |
Date: |
Sun, 10 Aug 2008 21:03:55 -0300 |
It was my idea to have all the real code in the staff-symbol file, and
have staff-symbol-referencer just be a wrapper, ie.
Staff_symbol_referencer::on_line(Grob *me, x)
{
Grob *st = staff_symbol(me);
return Staff_symbol::on_line(st, x)
}
(with an intelligent default if st == NULL)
On Sun, Aug 10, 2008 at 8:02 AM, Joe Neeman <address@hidden> wrote:
> Neil Puttock wrote:
>>
>> 2008/8/10 Joe Neeman <address@hidden>:
>>
>>> I think Han-Wen's point is that the first argument to
>>> Staff_symbol::some_function(Grob *me, ...) should always be a staff
>>> symbol (whereas the first argument to
>>> Staff_symbol_referencer::some_function(Grob *me, ...) should be a staff
>>> symbol referencer).
>>
>> That makes sense.
>>
>>> In other words, the old code was ok but it isn't ok for
>>> Staff_symbol::on_line to accept something that isn't a staff symbol.
>>
>> Since that's the case, what do you think of the revised patch I
>> posted, which moves all the code back to staff-symbol-referencer.cc?
>> We currently have the situation where 2.11.52-2 is broken for
>> augmentation dots and ties, so this needs sorting out. If you don't
>> think the patch is suitable, perhaps we should just revert the leger
>> line patch and revisit it later.
>
> Ah, sorry, I evidently hadn't read your last email completely. The patch
> you sent on the 7th looks pretty much OK to me. My only concern is that
> you use real numbers instead of ints in line-positions; do we really
> support that? If we do, it seems that dot-positions will be pretty
> messed up in that case.
>
> Also, do we have regression tests for all this stuff?
>
> Cheers,
> Joe
>
>
--
Han-Wen Nienhuys - address@hidden - http://www.xs4all.nl/~hanwen
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Neil Puttock, 2008/08/06
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Han-Wen Nienhuys, 2008/08/08
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Neil Puttock, 2008/08/09
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Joe Neeman, 2008/08/09
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Neil Puttock, 2008/08/10
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Joe Neeman, 2008/08/10
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Neil Puttock, 2008/08/10
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Joe Neeman, 2008/08/10
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves.,
Han-Wen Nienhuys <=
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Joe Neeman, 2008/08/10
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Neil Puttock, 2008/08/11
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Joe Neeman, 2008/08/12
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Neil Puttock, 2008/08/12
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Joe Neeman, 2008/08/12
- Re: [PATCH] Re: Duration dots and Bar lines on custom Staves., Han-Wen Nienhuys, 2008/08/12