Re: [PATCH] Implement new handling for margin and line-width settings.

2009-08-13 Thread Carl Sorensen
On 8/13/09 11:21 AM, "Michael Käppler" wrote: > Hi Carl, >>> Am I right you suggest to modify the output-def just once but permanent? >>> >> >> I meant to change the paper-def so that it would be consistent for the rest >> of the processing (it would eliminate multiple warnings). >> > O

Re: [PATCH] Implement new handling for margin and line-width settings.

2009-08-12 Thread David Kastrup
Carl Sorensen writes: > On 8/12/09 11:15 AM, "Michael Käppler" wrote: > >> Sorry, that wasn't intended. It was late yesterday and I didn't >> notice I pressed the wrong button... ;) > > Glad I'm not the only one who does this. Uh oh. Did you manage to placate her again? -- David Kastrup _

Re: [PATCH] Implement new handling for margin and line-width settings.

2009-08-12 Thread Carl Sorensen
On 8/12/09 11:15 AM, "Michael Käppler" wrote: > Hi Carl, >> Usually we like to keep responses to reviews on the -devel list, so I've >> copied this there. >> > Sorry, that wasn't intended. It was late yesterday and I didn't notice I > pressed the wrong button... ;) Glad I'm not the only one

Re: [PATCH] Implement new handling for margin and line-width settings.

2009-08-12 Thread Michael Käppler
Hi Carl, Usually we like to keep responses to reviews on the -devel list, so I've copied this there. Sorry, that wasn't intended. It was late yesterday and I didn't notice I pressed the wrong button... ;) Oh -- I misunderstood the meaning of the FIXME. I think I'd recommend changing the set

Re: [PATCH] Implement new handling for margin and line-width settings.

2009-08-11 Thread Carl Sorensen
Thanks for the reply Michael. Usually we like to keep responses to reviews on the -devel list, so I've copied this there. On 8/11/09 5:51 PM, "Michael Käppler" wrote: > Hi Carl, > thanks for your review: >> Don't use lmargin, rmargin, and lwidth as variable names. LilyPond >> standards c