Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(16948)

Issue 276570043: Dont emit \space glyph as empty page number (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
8 years, 4 months ago by pkx166h
Modified:
8 years, 4 months ago
Reviewers:
CC:
knut_petersen_t-online.de
Visibility:
Public.

Description

Dont emit \space glyph as empty page number Werner L reported in 2012 that LP emits a space glyph referencing the century schoolbook font at the top of every page. If you use gs (old and current) or pdfcrop to crop output with multiple pages and print-page-number = ##f there is excessive white space at the top of even pages. ghostscript should handle that better but a recent gs bug report http://bugs.ghostscript.com/show_bug.cgi?id=696445 was closed as "WONTFIX". There is a workaround within lilypond: \paper { [...] oddHeaderMarkup = \markup \null evenHeaderMarkup = \markup \null } Changing " " to "" cured the problem and appears to have no negative side effects. Bounding boxes are affected by this patch certainly so will have effects in situations where snippets are used by other third party programs.

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -4 lines) Patch
M ly/titling-init.ly View 2 chunks +2 lines, -4 lines 0 comments Download

Messages

Total messages: 4
pkx166h
Passes make, make check and a full make doc.
8 years, 4 months ago (2015-12-22 13:47:45 UTC) #1
pkx166h
Patch on countdown for December 28th.
8 years, 4 months ago (2015-12-25 12:13:52 UTC) #2
pkx166h
Patch counted down - please push. Knut, I'll push this for you. James
8 years, 4 months ago (2015-12-28 08:46:33 UTC) #3
pkx166h
8 years, 4 months ago (2015-12-28 09:05:28 UTC) #4
author	Knut Petersen <knut_petersen@t-online.de>	
	Tue, 22 Dec 2015 09:10:50 +0000 (09:10 +0000)
committer	James Lowe <pkx166h@gmail.com>	
	Mon, 28 Dec 2015 09:03:52 +0000 (09:03 +0000)
commit	d5bc965bceff7f08e1419bc73c039f395b4498d2
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b