[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Re: feature-request / doc-actualization (right-margin)
From: |
Neil Puttock |
Subject: |
Re: [PATCH] Re: feature-request / doc-actualization (right-margin) |
Date: |
Tue, 11 Aug 2009 23:56:36 +0100 |
2009/8/11 Michael Käppler <address@hidden>:
> here is the first draft of a patch concerning the margin settings for
> review.
Some comments:
+ "Checks whether @code{left-margin}, @code{right-margin}
and @code{line-width)"
{line-width}
This and the following docstring lines are too long.
+ Real paper_width = scm_to_double (lookup_variable (ly_symbol2scm
("paper-width")));
+ Real left_margin_default = scm_to_double (lookup_variable
(ly_symbol2scm ("left-margin-default")));
+ Real right_margin_default = scm_to_double (lookup_variable
(ly_symbol2scm ("right-margin-default")));
These are unsafe, since they return SCM_UNSPECIFIED instead of a Real
if the paper variables aren't defined; using robust_scm2double () here
will ensure a default is returned.
You can see the consequences of variable lookup failure by running
bookparts.ly
(http://git.savannah.gnu.org/gitweb/?p=lilypond.git;a=blob;f=input/regression/bookparts.ly;h=a814fb5e072911540ecf2ab44a7b7b09dae61835;hb=HEAD).
This snippet also fails to get a sane value for paper-height in
layout->page-init.
Regards,
Neil
- Re: feature-request / doc-actualization (right-margin), (continued)
- Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/08
- [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Mark Polesky, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Neil Puttock, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Mark Polesky, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Neil Puttock, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Mark Polesky, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/12
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin),
Neil Puttock <=
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/11
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/12
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Neil Puttock, 2009/08/13
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Joe Neeman, 2009/08/13
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/14
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/14
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Joe Neeman, 2009/08/15
- Message not available
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/16
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Joe Neeman, 2009/08/16
- Re: [PATCH] Re: feature-request / doc-actualization (right-margin), Michael Käppler, 2009/08/17