Re: rest/mm-rest-markup

2017-08-01 Thread David Kastrup
Carl Sorensen writes: > On 7/31/17 8:08 AM, "lilypond-devel on behalf of David Kastrup" > d...@gnu.org> wrote: > >> >>Now the thing is that with the new change in place, we would not >>necessarily _need_ different arguments: an integral multiplier larger >>than 1 could be taken as a multi-measur

Re: Enhance the -dpreview method for SVG output (issue 326960043 by pkx1...@gmail.com)

2017-08-01 Thread Étienne Beaulé
Hit send too soon... With the fix, there's still the run-time error: /home/ubuntu/lilypond/build/out/share/lilypond/current/scm/backend-library.scm:245:18: In procedure ly:paper-book-systems in expression (ly:paper-book-systems book): /home/ubuntu/lilypond/build/out/share/lilypond/current/scm/bac

Re: Enhance the -dpreview method for SVG output (issue 326960043 by pkx1...@gmail.com)

2017-08-01 Thread Étienne Beaulé
Now that solves half of the issue. The other half is with the width, as my patch would (should) also adjust the width to fit the music, leaving no whitespace on any side. This feature is basically only available when using \includes " lilypond-book-preamble.ly" of which only works with the eps bac

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread lemzwerg
I very much appreciate your insight and correction. So, there's no reason to provide or point to a separate list other than the Microsoft registry? Well, there is. As told before, the registry contains many, many features not relevant for Joe User. It would be beneficial if the most important

Re: OpenType info tools

2017-08-01 Thread Werner LEMBERG
> FontDrop is really handy! I can see myself using it all the time. > Although it seems to support many of the most likely used features, > it doesn't show them all. Note that you should be only interested in *typographic* features, not the features mandatory for complex script support. If you a

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread tisimst . lilypond
On 2017/08/01 17:49:47, lemzwerg wrote: > I looked into the latest Harfbuzz internals, and it looks like > the list of supported OpenType features is manually maintained. Nope. > Not all the features listed on the Microsoft feature registry > are supported, but many are. HarfBuzz is agno

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread lemzwerg
I looked into the latest Harfbuzz internals, and it looks like the list of supported OpenType features is manually maintained. Nope. Not all the features listed on the Microsoft feature registry are supported, but many are. HarfBuzz is agnostic to most features; if you select a feature, and

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread lemzwerg
If I'm not mistaken, you don't need to specify the script/language. That's part of the OpenType feature itself. In other words, if the feature is requested for a glyph outside the scripts/languages that the feature was specified for, the feature is not applied. Is that correct, Werner? No, it'

Re: segfault with wrong argument for dash-definition

2017-08-01 Thread Thomas Morley
2017-08-01 14:07 GMT+02:00 Carl Sorensen : > On 8/1/17 4:36 AM, "lilypond-devel on behalf of Thomas Morley" > thomasmorle...@gmail.com> wrote: > >>Hi, >> >>the following segfaults with any tested lily-version. >>{ \override Slur #'dash-definition = #'(0 1.0 0.5 0.5) c( c) } >>No other error-messag

Output better message for wrong argument of dash-definiton (issue 321400043 by thomasmorle...@gmail.com)

2017-08-01 Thread thomasmorley65
Reviewers: , Message: Please review Description: Output better message for wrong argument of dash-definiton Previously an argument like '(0 1.0 0.5 0.5) passed the pair?-typecheck, segfaulting later. Thus a new predicate, list-of-lists? is introduced. Please review this at https://codereview.a

Re: OpenType info tools

2017-08-01 Thread tisimst
On Tue, Aug 1, 2017 at 6:10 AM, Werner LEMBERG [via Lilypond] < ml+s1069038n204767...@n5.nabble.com> wrote: > > the following two tools should help any user identify scripts and > features of OpenType fonts. > > `fontdrop' is a nice browser tool which analyzes a font on your > harddisk without upl

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread tisimst . lilypond
On 2017/07/31 17:55:29, tisimst wrote: On 2017/07/31 17:48:10, lemzwerg wrote: > It's not Pango but HarfBuzz that processes (i.e., applies) the OpenType > features to a run of glyphs. Ah, I see! Thanks for clarifying that. I didn't realize Harfbuzz was part of the picture. Guess I haven't

Re: Let arpeggio-bracket/slur depend on grob-property thickness (issue 326970043 by thomasmorle...@gmail.com)

2017-08-01 Thread thomasmorley65
On 2017/08/01 12:53:25, Carl wrote: After reviewing the slur code, I remove my objections to using grob.line-thickness in this patch. ok, so I'll go for patch set 3 https://codereview.appspot.com/326970043/ ___ lilypond-devel mailing list lilypond-d

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread tisimst . lilypond
On 2017/08/01 15:08:11, trueroad wrote: On 2017/08/01 15:05:46, trueroad wrote: > Add notes and identification way If I understand correctly, I've found a current limitation. I could not find a way to specify OpenType font scripts and languages. So I've added it to the document. Also I've a

Re: Add usage of OpenType font feature to the documents (issue 328140043 by truer...@gmail.com)

2017-08-01 Thread trueroad
On 2017/08/01 15:05:46, trueroad wrote: Add notes and identification way If I understand correctly, I've found a current limitation. I could not find a way to specify OpenType font scripts and languages. So I've added it to the document. Also I've added feature identification way etc. https:

Re: rest/mm-rest-markup

2017-08-01 Thread Carl Sorensen
On 7/31/17 8:08 AM, "lilypond-devel on behalf of David Kastrup" wrote: > >Now the thing is that with the new change in place, we would not >necessarily _need_ different arguments: an integral multiplier larger >than 1 could be taken as a multi-measure rest count, like > >{1*4} > >So we likely _co

Re: Enhance the -dpreview method for SVG output (issue 326960043 by pkx1...@gmail.com)

2017-08-01 Thread Paul
On 07/27/2017 03:21 AM, pkx1...@gmail.com wrote: This is a first attempt to merge the dump-page and dump-preview methods so that there is an option for cropping pages that are not just previews. I wonder... is the desired functionality already provided by one-page-breaking and/or one-line-aut

Re: Let arpeggio-bracket/slur depend on grob-property thickness (issue 326970043 by thomasmorle...@gmail.com)

2017-08-01 Thread Carl . D . Sorensen
After reviewing the slur code, I remove my objections to using grob.line-thickness in this patch. https://codereview.appspot.com/326970043/ ___ lilypond-devel mailing list lilypond-devel@gnu.org https://lists.gnu.org/mailman/listinfo/lilypond-devel

Re: Let arpeggio-bracket/slur depend on grob-property thickness (issue 326970043 by thomasmorle...@gmail.com)

2017-08-01 Thread Carl . D . Sorensen
I believe that line-thickness should not be changed for the grob; just thickness. line-thickness should be the staff line-thickness, not changed per grob, IMO. https://codereview.appspot.com/326970043/diff/40001/lily/arpeggio.cc File lily/arpeggio.cc (right): https://codereview.appspot.com/326

Re: midi2ly on mac: midi.so wrong architecture

2017-08-01 Thread Carl Sorensen
On 7/28/17 3:09 PM, "lilypond-devel on behalf of ElRay" wrote: >>> I am pretty sure that Graham posted a patch for this on one of the >>>lists >>> one of the last times this was being discussed. > >Tracking on that, but it's been 5 years since the ticket was opened and >the >patch hasn't made it

Re: Enhance the -dpreview method for SVG output (issue 326960043 by pkx1...@gmail.com)

2017-08-01 Thread Carl . D . Sorensen
Oh, now I read the first message and see that the code doesn't work. My first attempt at getting the code to work would be to change "stensil" to "stencil" and see if that fixed things. https://codereview.appspot.com/326960043/ ___ lilypond-devel mai

Re: Enhance the -dpreview method for SVG output (issue 326960043 by pkx1...@gmail.com)

2017-08-01 Thread Carl . D . Sorensen
I have listed some specific changes that must be made (move to the proper alphabetical order) and raised questions about how the code works with an apparently misspelled argument. I believe both of those need to be fixed before pushing the patch. I would also like to see crop apply to at least

OpenType info tools

2017-08-01 Thread Werner LEMBERG
Folks, the following two tools should help any user identify scripts and features of OpenType fonts. `fontdrop' is a nice browser tool which analyzes a font on your harddisk without uploading it to a place in the internet. It needs a recent browser that supports OpenType CSS features. [I haven

segfault with wrong argument for dash-definition

2017-08-01 Thread Thomas Morley
Hi, the following segfaults with any tested lily-version. { \override Slur #'dash-definition = #'(0 1.0 0.5 0.5) c( c) } No other error-message than: segmentation fault (core dumped) Wheras { \override Slur #'dash-definition = #"foo" c( c) } returns a meaningful message, a warning not even an err

PATCHES - Countdown for August 1st

2017-08-01 Thread James
Hello, Here is the current patch countdown list. The next countdown will be on August 4th. A quick synopsis of all patches currently in the review process can be found here: http://philholmes.net/lilypond/allura/ Push: 5166 Doc: Delete note for svg backend which become unnecessary - Mas

Re: Enhance the -dpreview method for SVG output (issue 326960043 by pkx1...@gmail.com)

2017-08-01 Thread pkx166h
I am going to leave this on review for this countdown only that the submitter (not me) had doubts about this patch and was looking for any guidance (it does pass all the tests BTW). https://codereview.appspot.com/326960043/ ___ lilypond-devel mailing l