Thank you for your reviewing.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely
File Documentation/usage/running.itely (right):

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode652
Documentation/usage/running.itely:652: This is useful when you want to
create a font-non-embedded PDFs first
On 2020/04/28 15:51:33, lemzwerg wrote:
> Replace this and the next line with
> 
> This is useful if you want to create PDFs without embedded fonts
first, letting
> Ghostscript embed the fonts in a follow-up step as shown below.

I used the sentence Trevor suggested below.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode662
Documentation/usage/running.itely:662: Note: Unlikely
@code{font-ps-resdir},
On 2020/04/28 17:37:01, Trevor Daniels wrote:
> Unlike ...

Done.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode668
Documentation/usage/running.itely:668: because embedding TrueType fonts
later cause garbled characters.
On 2020/04/28 17:37:01, Trevor Daniels wrote:
> ... causes ...

Done.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode670
Documentation/usage/running.itely:670: @code{gs-never-embed-fonts}
embeds TrueType fonts despite its name.
On 2020/04/28 17:37:00, Trevor Daniels wrote:
> To avoid garbling characters, use
> @code{gs-never-embed-fonts}, as this embeds TrueType fonts
> despite its name.

Done.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode679
Documentation/usage/running.itely:679: and then embed the fonts with
Ghostscript as shown below.
On 2020/04/28 17:37:01, Trevor Daniels wrote:
> @item @code{font-ps-resdir} = @var{directory}
> Set @var{directory} to build a subset of the PostScript resource
directory to be
> used for embedding fonts later.
> This is useful when you want to create a PDF without embedded
> fonts first and later embed the fonts with Ghostscript as shown below.

I'd like to use @var{string} instead of @var{directory}, so I propose
this sentence.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode694
Documentation/usage/running.itely:694: Note: Unlikely
@code{font-export-dir},
On 2020/04/28 17:37:01, Trevor Daniels wrote:
> Unlike ...

Done.

https://codereview.appspot.com/577900043/diff/577910043/Documentation/usage/running.itely#newcode702
Documentation/usage/running.itely:702: @code{gs-never-embed-fonts}
embeds TrueType fonts despite its name.
On 2020/04/28 17:37:01, Trevor Daniels wrote:
> See corresponding change above.

Done.

https://codereview.appspot.com/577900043/diff/577910043/scm/framework-ps.scm
File scm/framework-ps.scm (right):

https://codereview.appspot.com/577900043/diff/577910043/scm/framework-ps.scm#newcode291
scm/framework-ps.scm:291: (ly:warning (_ "Font ~a cannot be used in
PostScript resource directory because it is an OpenType/CFF Collection
(OTC) font.") name))
On 2020/04/28 15:51:33, lemzwerg wrote:
> Please fold lines longer than 80 characters.

Done.

https://codereview.appspot.com/577900043/

Reply via email to