Thank you for your reviewing.

https://codereview.appspot.com/548080043/diff/567530058/lily/open-type-font-scheme.cc
File lily/open-type-font-scheme.cc (right):

https://codereview.appspot.com/548080043/diff/567530058/lily/open-type-font-scheme.cc#newcode467
lily/open-type-font-scheme.cc:467: warning (_f ("font index %d too large
for font `%s', using index 0",
On 2020/05/04 04:33:59, lemzwerg wrote:
> Maybe it makes sense to use a format like
> 
>   font `%s': ...
> 
> here, too.

Done.

https://codereview.appspot.com/548080043/diff/567530058/lily/open-type-font-scheme.cc#newcode495
lily/open-type-font-scheme.cc:495: warning (_f ("font `%s': cannot read
`%s' field of subfont %d",
On 2020/05/04 04:33:59, lemzwerg wrote:
> Shall we do
> 
>   s/`%s' field/field `%s'/
> 
> here and in the following messages?  Ditto
> 
>   s/`%s' table/table `%s'/
> 
> later on.  We already have `font xxx' and `subfont yyy'...

Done.

https://codereview.appspot.com/548080043/diff/567530058/lily/open-type-font-scheme.cc#newcode513
lily/open-type-font-scheme.cc:513: warning (_f ("font `%s': cannot open
for writing",
On 2020/05/04 04:33:59, lemzwerg wrote:
> Shall we replace `font' with `subfont' here and in succeeding write
messages?

Done.

https://codereview.appspot.com/548080043/diff/567530058/lily/open-type-font-scheme.cc#newcode594
lily/open-type-font-scheme.cc:594: " table no. %u of subfont %d",
On 2020/05/04 04:33:59, lemzwerg wrote:
> s/table no. %u/table %u/

Done.

https://codereview.appspot.com/548080043/

Reply via email to