tdanielsmusic | 4 Jul 16:21 2015

Python-ly to indent ../input/regression/a* (issue 249270044 by pkx166h <at> gmail.com)

On balance the indentation is improved, but there are a few cases which
are worse, IMHO.  With a follow-up hand-indentation to "uncorrect" the
worst cases this would be worth doing, if just to remove the tabs.

Trevor

https://codereview.appspot.com/249270044/diff/1/input/regression/alignment-vertical-manual-setting.ly
File input/regression/alignment-vertical-manual-setting.ly (right):

https://codereview.appspot.com/249270044/diff/1/input/regression/alignment-vertical-manual-setting.ly#newcode3
input/regression/alignment-vertical-manual-setting.ly:3: texidoc =
"Alignments may be changed per system by setting
The texidoc string formatting is rather inconsistent.

https://codereview.appspot.com/249270044/diff/1/input/regression/alter-broken.ly
File input/regression/alter-broken.ly (right):

https://codereview.appspot.com/249270044/diff/1/input/regression/alter-broken.ly#newcode32
input/regression/alter-broken.ly:32: (ly:stencil-rotate
(ly:hairpin::print grob) -5 0 0)))
Definitely don't like this

https://codereview.appspot.com/249270044/diff/1/input/regression/alter-broken.ly#newcode42
input/regression/alter-broken.ly:42: ) Tie
This does not seem right for either Scheme or Lily.

https://codereview.appspot.com/249270044/diff/1/input/regression/auto-beam-exceptions.ly
File input/regression/auto-beam-exceptions.ly (right):

https://codereview.appspot.com/249270044/diff/1/input/regression/auto-beam-exceptions.ly#newcode29
(Continue reading)

k-ohara5a5a | 4 Jul 05:59 2015
Picon

Doc: Issue 4059: Document MIDI mapping and MIDI effects (issue 249980043 by tdanielsmusic <at> googlemail.com)

Sorry I'm late.
You might still slip in a correction, or just remove mention of
midiChannelMapping='voice since we haven't found a use for it.

https://codereview.appspot.com/249980043/diff/1/Documentation/notation/input.itely
File Documentation/notation/input.itely (right):

https://codereview.appspot.com/249980043/diff/1/Documentation/notation/input.itely#newcode3279
Documentation/notation/input.itely:3279:  <at> ref{MIDI channel mapping}).
This is the part that doesn't work.  We could say
"the MIDI channel associated with the current Staff (or the current the
Voice if you have moved the Staff_performer to the Voice context).  If
you set midiChannelMapping to 'instrument, all staves with the same
instrument are affected when you set one of these context properties.
Setting midiChannelMapping to 'voice often prevents the context
properties form having effect."

https://codereview.appspot.com/249980043/
nine.fierce.ballads | 4 Jul 03:47 2015
Picon

Issue 4478: Disentangle some Tie and Semi_tie code. (issue 251840044 by nine.fierce.ballads <at> gmail.com)

Reviewers: ,

Description:
Both are Grobs, but ties are Spanners and semi-ties are Items.  The code
for both cases was jumbled together in many Tie methods.  I hope that I
have clarified a few things.  No functional changes are intended.

Please review this at https://codereview.appspot.com/251840044/

Affected files (+111, -87 lines):
   M lily/completion-note-heads-engraver.cc
   M lily/include/semi-tie.hh
   M lily/include/tie.hh
   M lily/include/tie-column.hh
   M lily/semi-tie.cc
   M lily/tie.cc
   M lily/tie-column.cc
   M lily/tie-engraver.cc
   M lily/tie-formatting-problem.cc
   M lily/tie-specification.cc
thomasmorley65 | 3 Jul 12:29 2015
Picon

Re: Avoid treating x as relative in \relative { \autochange {x} } (issue 250170043 by nine.fierce.ballads <at> gmail.com)

Looks like it's now consistent with the NR saying:
"A \relative section that is outside of \autochange has no effect on the
pitches of the music, so if necessary, put \relative inside \autochange.
"

LGTM

https://codereview.appspot.com/250170043/
James | 2 Jul 12:46 2015
Picon

PATCHES: Countdown for July 5th 2015

Hello,

Here is the current patch countdown list. The next countdown will be on
July 5th.

You can always view the most current countdown list here:
http://code.google.com/p/lilypond/issues/list?q=Patch%3Apush%2Ccountdown%2Creview%2Cnew%2Cwaiting&colspec=Patch%20Owner%20ID%20Summary&sort=patch

____________________

PUSH:

Dan Eble: Auto_change_iterator: move staff creation to scheme
http://code.google.com/p/lilypond/issues/detail?id=4465

David Kastrup: Create a module variable access system for C++
http://code.google.com/p/lilypond/issues/detail?id=4462

Trevor Daniels: Documentation for issues 3601 (based on 3581), 4042 (NR:
"MIDI output", Section 3.5)
http://code.google.com/p/lilypond/issues/detail?id=4059

____________________

COUNTDOWN:

David Kastrup: event-chord-wrap! throws syntax error
http://code.google.com/p/lilypond/issues/detail?id=4471

David Kastrup: freetype.cc: solve algebra puzzle for converting
(Continue reading)

Walter | 30 Jun 21:04 2015
Picon

Lilypond output to Libreoffice 5.0

Hi,

I have been using OOoLilypond with Libreoffice for quite some time. I have
tested the extension with LibreofficeDev 5.0 and the following error showed
up: "com.star.comp.Writer.TextDocument: document type not supported by
OOoLilypond" (System: Win 7 x64).

It seems that the API has changed and the extension will become obsolete. I
would greatly appreciate if you could release another version of the
OOoLilypond extension.

Otherwise you will have to update this webpage:
http://www.lilypond.org/doc/v2.19/Documentation/usage/lilypond-output-in-other-programs#inserting-lilypond-output-into-openoffice-and-libreoffice

Thanks
Walter
Urs Liska | 1 Jul 09:24 2015

Re: interactive sheet music experiment


Am 01.07.2015 um 04:02 schrieb Andrew Levine:
> Hello -- I recently made a novel use of Lilypond by using Javascript to
> place English letters underneath notes in the SVG output, and allowing the
> user to play the song in an online game called Piano Typewriter. Is there
> any lists I can send this link to other than here?
> 
> http://www.picatino.com/piano_typewriter/

This looks interesting.
However, I'm not really sure what you want with it (or probably there's
an issue with what the site currently does).

What I see after choosing a song is that seemingly arbitrary letters in
a  Courier-like font are placed above the noteheads. If these letters
should be the note names then it seems to be not working properly. If
these should *not* be note names I'm dearly missing an
explanation/introduction on the website.

Additionally I'd be interested to learn how the actual engraving is
achieved. Obviously you are importing something from Mutopia. Is that
the plain lily files or already some engraved material? I'm asking
because a few random checks show that some scores are engraved rather
poorly. If you are importing LilyPond sources from Mutopia you might
have to check if they are engraved properly with the version of LilyPond
you are using. For example have a look at Bach's Goldberg Variation 9
(Can't provide a link because it's JavaScript).

Please take this not as a criticism of your idea but as constructive hints.
Once it is working properly you should post that to the lilypond-user
(Continue reading)

Andrew Levine | 1 Jul 04:02 2015
Picon

interactive sheet music experiment

Hello -- I recently made a novel use of Lilypond by using Javascript to
place English letters underneath notes in the SVG output, and allowing the
user to play the song in an online game called Piano Typewriter. Is there
any lists I can send this link to other than here?

http://www.picatino.com/piano_typewriter/
lemzwerg | 30 Jun 16:00 2015

freetype.cc: solve algebra puzzle for converting quadratics to cubics (issue 252810043 by dak <at> gnu.org)

LGTM.  Funny that noone has every tried to simplify this...

https://codereview.appspot.com/252810043/
MacUpdate | 30 Jun 15:12 2015

MacUpdate - your app listing has been updated


                           [1][macupdate-logo.png]
                               top background

                            App Listing Updated

   Hi lilypond.org, We have updated your application listing for LilyPond
   2.19.22-1 on [2]MacUpdate.com. Please take a moment to review your
   application's information to make sure that everything is correct.

   [3]View your updated app listing view your updated app
    You can [4]login to your Developer Account to reply to comments and
   view stats, or [5]submit new updates and changes to your app listing.

                 [6]MacUpdate Desktop install compatibility

   Desktop Install Compatibility

   MacUpdate Desktop 6 is helping developers make it easier for users to
   fully install and use their apps. Download Desktop 6 and to ensure your
   app works with the "Install" link on our download pages.

   Advertise With MacUpdate

      The best Mac devs advertise their apps on MacUpdate.com because it's
      the most targeted, highly performing Mac app advertising you can find
      on the web. Contact ads <at> macupdate.com to guarantee your annual ad
      campaigns get booked and expand your app's audience.

   Learn more view your updated app
(Continue reading)

lemzwerg | 30 Jun 06:39 2015

Re: stencil-integrate.cc: root out "slopes" (issue 246590043 by dak <at> gnu.org)

LGTM

https://codereview.appspot.com/246590043/diff/20001/lily/stencil-integral.cc
File lily/stencil-integral.cc (right):

https://codereview.appspot.com/246590043/diff/20001/lily/stencil-integral.cc#newcode62
lily/stencil-integral.cc:62: void create_path_cap (vector<Box> &boxes,
vector<Drul_array<Offset> > &buildings, PangoMatrix trans, Offset pt,
Real rad, Offset dir);
80 characters per line, please...

https://codereview.appspot.com/246590043/diff/20001/lily/stencil-integral.cc#newcode363
lily/stencil-integral.cc:363: create_path_cap (vector<Box> &boxes,
vector<Drul_array<Offset> > &buildings, PangoMatrix trans, Offset pt,
Real rad, Offset dir)
ditto

https://codereview.appspot.com/246590043/

Gmane