Sorry this took so long. I uploaded all 6 commits as separate patch sets so ...
14 years, 6 months ago
(2010-10-14 06:53:45 UTC)
#1
Sorry this took so long. I uploaded all 6 commits
as separate patch sets so it's easier to review.
Patch set 7 combines all the changes into one set.
Okay to push?
- Mark
Would anyone like to look over this or shall I just push it? It compiles ...
14 years, 6 months ago
(2010-10-16 16:00:27 UTC)
#2
Would anyone like to look over this or shall
I just push it? It compiles fine, but I just
assumed someone would want to double-check
it, considering all the debate this topic has
generated recently.
- Mark
Looks ok to me, although I haven't tried committing it. Let's push it and move ...
14 years, 6 months ago
(2010-10-16 20:38:22 UTC)
#3
Looks ok to me, although I haven't tried committing it.
Let's push it and move on to trying it out. That
might generate more comment.
BTW, I don't see these (old) \paper block variables in
the IR. Are they not there or have I missed them?
It would be useful for anyone starting to adjust
these to know what the default values are, or maybe
more accurately how they are derived.
Trevor
Issue 2505041: vertical spacing: Rename dimensions.
(Closed)
Created 14 years, 6 months ago by Mark Polesky
Modified 14 years, 6 months ago
Reviewers: Trevor Daniels, Graham Percival (old account)
Base URL:
Comments: 0