On 2011/09/17 19:29:40, janek wrote:
It is ready to go except for the above.
Greetings,
I realize that this patch has been sort of forgotten, so I thought I’d
update it and re-upload it:
https://codereview.appspot.com/227070043/
https://codereview.appspot.com/4636081/
_
Needs a regression test exercising the different settings.
http://codereview.appspot.com/4636081/diff/38001/lily/stem-tremolo.cc
File lily/stem-tremolo.cc (right):
http://codereview.appspot.com/4636081/diff/38001/lily/stem-tremolo.cc#newcode118
lily/stem-tremolo.cc:118: shape = ly_symbol2scm ("
Oh no! I hoped to get it on this countdown.
A version compatible with fixcc uploaded.
http://codereview.appspot.com/4636081/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
cannot apply to master due to fixcc.
http://codereview.appspot.com/4636081/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
make passes and no reg tests differences
http://codereview.appspot.com/4636081/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
Reinhold:
ok, replaced parallelogram with beam-like
http://codereview.appspot.com/4636081/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
Am Freitag 29 Juli 2011, 07:38:26 schrieben Sie:
> New patch set uploaded. Minor style change: slash shape that was named
> "default" (which was confusing) is now named "parallelogram" (because
> that's how it looks like).
> If you don't like parallelograms ;) we can name it beam-like or sth.
Yes
New patch set uploaded. Minor style change: slash shape that was named
"default" (which was confusing) is now named "parallelogram" (because
that's how it looks like).
If you don't like parallelograms ;) we can name it beam-like or sth.
http://codereview.appspot.com/4636081/
___
Joe: redundant lines removed.
Interesting thing happens when i compare regtests: i see a difference in
part-combine-tuplet-end.ly, which is completely unrelated to tremolos.
Even funnier, my branch compiled this test better than master...
Perhaps i should've built from scratch.
BTW, should i add
makes fine. No reg test differences now.
http://codereview.appspot.com/4636081/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/lilypond-devel
http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc
File lily/stem-tremolo.cc (right):
http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc#newcode93
lily/stem-tremolo.cc:93: style = ly_symbol2scm ("constant");
On 2011/07/23 19:58:19, Janek Warchol wrote:
On 2
New patch set uploaded. Current Lily default behaviour is kept as
default now.
http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc
File lily/stem-tremolo.cc (right):
http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc#newcode42
lily/stem-tremolo.cc:42: style
http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc
File lily/stem-tremolo.cc (right):
http://codereview.appspot.com/4636081/diff/10001/lily/stem-tremolo.cc#newcode42
lily/stem-tremolo.cc:42: style = ly_symbol2scm ("constant");
You can remove these two lines and use
style == ly
2011/7/19 :
> Make works now and I get some reg test output differences
> http://code.google.com/p/lilypond/issues/detail?id=1735#c4
They look exacly like they are expected to look.
thanks,
Janek
___
lilypond-devel mailing list
lilypond-devel@gnu.org
Make works now and I get some reg test output differences
See
http://code.google.com/p/lilypond/issues/detail?id=1735#c4
James
http://codereview.appspot.com/4636081/
___
lilypond-devel mailing list
lilypond-devel@gnu.org
https://lists.gnu.org/mailma
2011/7/18 Neil Puttock :
> 2011/7/18 Janek Warchoł :
>
>> Btw,
>>
>>> [/home/jlowe/lilypond-git/build/out/share/lilypond/current/scm/document-markup.scm]
>>> Writing "internals.texi"...ERROR: In procedure procedure-name:
>>> ERROR: Wrong type argument in position 1: #f
>>> make[1]: *** [out/interna
2011/7/18 Janek Warchoł :
> Btw,
>
>> [/home/jlowe/lilypond-git/build/out/share/lilypond/current/scm/document-markup.scm]
>> Writing "internals.texi"...ERROR: In procedure procedure-name:
>> ERROR: Wrong type argument in position 1: #f
>> make[1]: *** [out/internals.texi] Error 1
>> rm out/weblink
2011/7/18 :
> Hello, can someone else verify?
>
> I get a make fail:
I get it too... Strange, it compiled some time ago.
Btw,
> [/home/jlowe/lilypond-git/build/out/share/lilypond/current/scm/document-markup.scm]
> Writing "internals.texi"...ERROR: In procedure procedure-name:
> ERROR: Wrong type
Hello, can someone else verify?
I get a make fail:
--snip--
[/home/jlowe/lilypond-git/build/out/share/lilypond/current/scm/document-markup.scm]
Writing "internals.texi"...ERROR: In procedure procedure-name:
ERROR: Wrong type argument in position 1: #f
make[1]: *** [out/internals.texi] Error 1
r
Finally - new patch set uploaded.
Now it's possible to easily switch between different tremolo behaviours.
'Style' property is no longer used to choose between rectangular and
beam-like slashes - this is now done using 'shape' property.
'Style' property now influences both 'shape' and 'slope' of t
Actually, non-rectangular, constant-sloped beams used to be the default,
but I changed them (some years ago now) to be rectangular and parallel
to the beam, since that's what most of my scores have. My music
collection isn't with me right now, but I can confirm at least that
B&H's urtext edition o
Hi Janek,
I'm afraid I don't like this at all. While the authorities may be in
agreement that the slashes should be sloped, all the scores I've looked
at follow the same style as LilyPond (which I think reflects a more
traditional hand-engraved style).
It would be preferable to allow users to c
Looks mostly good to me.
Thanks,
Carl
http://codereview.appspot.com/4636081/diff/1/lily/stem-tremolo.cc
File lily/stem-tremolo.cc (right):
http://codereview.appspot.com/4636081/diff/1/lily/stem-tremolo.cc#newcode42
lily/stem-tremolo.cc:42: that it's not proper notation.
We shouldn't keep the
23 matches
Mail list logo
