[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: lilypond-book: Group line-width settings together (issue 2222). (iss
From: |
reinhold . kainhofer |
Subject: |
Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056) |
Date: |
Tue, 24 Jan 2012 17:06:57 +0000 |
http://codereview.appspot.com/5553056/diff/1/python/book_snippets.py
File python/book_snippets.py (left):
http://codereview.appspot.com/5553056/diff/1/python/book_snippets.py#oldcode806
python/book_snippets.py:806: self.ext = os.path.splitext
(os.path.basename (self.filename))[1]
On 2012/01/22 00:08:45, Julien Rioux wrote:
The '.ly' is hardcoded elsewhere already, so that when you
include a .ily, lilypond-book actually writes a lily-????.ly file for
it, and
this is what we link to. On the other hand leaving the current code in
leads to
broken links to a lily-????.ily file.
What about --use-source-file-names?
http://codereview.appspot.com/5553056/diff/1/python/book_snippets.py
File python/book_snippets.py (right):
http://codereview.appspot.com/5553056/diff/1/python/book_snippets.py#newcode124
python/book_snippets.py:124: line-width = #(- line-width (* mm
%(padding_mm)f) (* mm 1))''',
On 2012/01/22 00:08:45, Julien Rioux wrote:
On 2012/01/21 23:54:36, Reinhold wrote:
> Aren't there cases when neither LINE_WIDTH nor QUOTE is used?
Yes, that's the case for html-inline-no-option.html
I was rather talking about e.g. when the papersize=... snippet option is
used in a texi file... (I think that doesn't work currently, either,
though).
> In that case we don't subtract the padding...
Why would a padding be relevant when there is no specified line width?
The line width is implicitly specified by the paper size.
http://codereview.appspot.com/5553056/
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), (continued)
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), graham, 2012/01/20
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), janek . lilypond, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), graham, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), julien . rioux, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), janek . lilypond, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), janek . lilypond, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), reinhold . kainhofer, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), julien . rioux, 2012/01/21
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056),
reinhold . kainhofer <=
- Re: lilypond-book: Group line-width settings together (issue 2222). (issue 5553056), julien . rioux, 2012/01/24