[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Allows slurs to break at barlines. (issue 7424049)
From: |
address@hidden |
Subject: |
Re: Allows slurs to break at barlines. (issue 7424049) |
Date: |
Tue, 9 Apr 2013 22:59:57 +0300 |
On 9 avr. 2013, at 20:31, address@hidden wrote:
> On 2013/03/26 04:10:38, mike7 wrote:
>> >> On 25 mars 2013, at 07:10, mailto:address@hidden wrote:
>> >>
>> >>> Please do just one manual review of the regression suite between
>> >>> versions before adding another test of this length.
>> >>
>> I wanted to exhaust as many possibilities as I could think of (broken
> slurs over
>> one note, several notes, beginning broken, and broken, etc.) for both
> slur and
>> phrasing. Would it be better to split each system up into a separate
> test? Get
>> rid of some tests?
>
> We need exhaustive test files while debugging, but probably we should
> reduce them to the essentials for the regtest that gets pushed.
>
> Issue 3307 was plain to see in the regtest
> ‘rest-on-nonstandard-staff.ly’, but nobody succeeded in seeing the
> problem.
>
Ok - feel free to chop this down however you see fit - I trust you.
Cheers,
MS
> https://codereview.appspot.com/7424049/