Re: [PATCH] Fix issue 778

2009-06-19 Thread Neil Puttock
2009/6/18 Patrick McCarty : > Okay, thanks for the guidance.  Attached is a revised patch. Cheers, it's applied. Regards, Neil ___ lilypond-devel mailing list lilypond-devel@gnu.org http://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: [PATCH] Fix issue 778

2009-06-18 Thread Patrick McCarty
On Wed, Jun 17, 2009 at 10:41:24PM +0100, Neil Puttock wrote: > 2009/6/17 Han-Wen Nienhuys : > > I think this can be neater.  By tracking the 'cause property > > backwards, this could simply be a grob property callback, and the C++ > > does not need to touch the > > 'dot-count property. > > We alr

Re: [PATCH] Fix issue 778

2009-06-17 Thread Neil Puttock
2009/6/17 Han-Wen Nienhuys : > I think this can be neater.  By tracking the 'cause property > backwards, this could simply be a grob property callback, and the C++ > does not need to touch the > 'dot-count property. We already have the callback, as I pointed out in the other thread: dots::calc-dot

Re: [PATCH] Fix issue 778

2009-06-16 Thread Han-Wen Nienhuys
I think this can be neater. By tracking the 'cause property backwards, this could simply be a grob property callback, and the C++ does not need to touch the 'dot-count property. See for example note-head::calc-duration-log On Tue, Jun 16, 2009 at 7:58 PM, Patrick McCarty wrote: > This patch fixe

[PATCH] Fix issue 778

2009-06-16 Thread Patrick McCarty
Hello, This patch fixes issue 778 in the tracker. If I should add a regression test, I can do that. Thanks, Patrick >From 5fa8f35cfeb215ba97af7a89cd77c5c31393fff8 Mon Sep 17 00:00:00 2001 From: Patrick McCarty Date: Mon, 15 Jun 2009 18:35:47 -0700 Subject: [PATCH] Fix #778 * Read the default v