[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #5449 Chord names clean-up; no more Banter, exceptionsPartial or \powerChords.

[Auto mailings of changes to Lily Issues via Testlilyissues-auto]

• Patch: countdown --> push
• Comment:

Patch counted down - please push.

---

[issues:#5449] Chord names clean-up; no more Banter, exceptionsPartial or \powerChords.

Status: Started
Created: Mon Dec 17, 2018 05:23 PM UTC by Valentin Villenave
Last Updated: Fri Jan 11, 2019 10:59 AM UTC
Owner: Valentin Villenave
Attachments:

• test-results_5449_100119.zip (331.4 kB; application/x-zip-compressed)

Chord names clean-up; no more Banter, exceptionsPartial or \powerChords.

This rather large patch

• removes the legacy code for unsupported non-Ignatzek chord names;
  that code hasn’t been working properly for nearly 15 years anyway, see
  https://lists.gnu.org/archive/html/lilypond-user/2008-08/msg00288.html

• stores a version of this code, with minor corrections, as a
  self-contained snippet (for historical purposes mainly), both
  on the LSR (as an upgrade to snippet #102) and in a dedicated
  subsection of the Snippets doc page.

• as a result, chord-generic-names.scm is no longer found in scm/
  and the chord-names-jazz.ly doc chart no longer includes any
  alternative chord naming systems (that were already broken anyway).

• chordNamesExceptionsFull and chordNamesExceptionsPartial
  properties have been removed as well (both having no longer had
  any effect whatsoever for quite some time).

• the main (and now only) chordNamesExceptions property def
  has been slightly updated for slightly neater glyphs and spacing,
  and so as to now include power chords by default; as a result,
  \powerChords is no longer needed, thereby solving any possible
  confusion with \germanChords and the like. (I can’t think of any
  case where users would explicitely type \chords{c:5} in order to
  get the same output as \chords{c} -- can you?)

• the documentation has been updated as well; convert rules
  have been added (although these may be merged with previous
  2.21.0 rules once the release cycle gets back on track).

http://codereview.appspot.com/363880043

---

Sent from sourceforge.net because address@hidden is subscribed to https://sourceforge.net/p/testlilyissues/issues/

To unsubscribe from further messages, a project admin can change settings at https://sourceforge.net/p/testlilyissues/admin/issues/options. Or, if this is a mailing list, you can unsubscribe from the mailing list.

_______________________________________________
Testlilyissues-auto mailing list
address@hidden
https://lists.sourceforge.net/lists/listinfo/testlilyissues-auto