[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Gets vertical skylines from grob stencils (issue 5626052)

From: mtsolo
Subject: Re: Gets vertical skylines from grob stencils (issue 5626052)
Date: Thu, 16 Feb 2012 10:08:39 +0000
File lily/ (right):
lily/ vector<Grob *> *riders)
On 2012/02/16 08:09:10, joeneeman wrote:
I don't understand why riders are necessary. Is it because of this
dependence stuff?

Not cyclical, but imagine that a grob (say tuplet bracket, for example)
has its outside staff priority set whereas one of its Y_AXIS children
(say tuplet number, for example), doesn't.  If the tuplet number's
skyline is added to the skyline_pair using add boxes before its parent
is shifted, it will be placed too low in the skyline.  Thus, it must be
added to the skyline only after its parent's outside-staff-priority has
been set to SCM_BOOL_F.  This is why riders are used.
File lily/ (right):
lily/ Strips all old sloped material, adds new.
On 2012/02/16 08:09:10, joeneeman wrote:
You're assuming that all sloped material came from skyline padding.
That's true
right now, but there's no reason that it will continue to be true.
It's probably
better to avoid adding padding at creation time altogether, and
instead to add
it when calling Skyline::distance.

Right you are...I agree that this is entirely dependent on the
implementation of skylines at the moment.  I know that I always respond
to you that it's something that can be done in a future patch, but I'd
put this in that category as well, as it seems like a separate problem
that doesn't need to be tacked on to this patch to be solved.  That
said, if I forget, nag me!
lily/ Skyline::left () const
On 2012/02/16 08:09:10, joeneeman wrote:
This is linear in the number of buildings, but it should be constant.

How would one do this?
lily/ Skyline::right () const
On 2012/02/16 08:09:10, joeneeman wrote:

File lily/ (right):
lily/ SCM glyph_info = scm_hashq_ref (font_list,
scm_string_to_symbol (glyph), SCM_EOL);
On 2012/02/16 08:09:10, joeneeman wrote:
There isn't much error-checking here. What if the user substitutes an
font which isn't in the list?

By the way, lilypond's fonts embed extra data in font tables (look for
LILC and
LILY in That may be a better way to embed this
than by putting it in a scm file. For example, it would allow
unofficial fonts
to support integrals by embedding an extra table.

I think this is a good idea...I'll have a look at it.  Question, though
- aren't font tables done as alists?  I think it's important to use a
hash table here, as there is a lot of data getting thrown around and
hash table look ups are in constant time.  I'll investigate.
File mf/GNUmakefile (right):
mf/GNUmakefile:231: $(LILYPOND_BINARY) --verbose
$(top-src-dir)/ly/generate-font-integrals > $@
On 2012/02/15 10:26:17, Graham Percival wrote:
This looks good, but not immediately relevant to the rest of the
skylines stuff.  Could you make this a separate patch so that it can
be pushed

Sorry for not responding to this comment inlined...the response is that
it'd be hard as it requires new files and/or changes to old ones to go
along w/ it.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]