Fix micro-tones assigned to strings in TabStaff

Issue Tracker for LilyPond

Brought to you by: fedelibre, hahnjo, philholmes, tdanielsmusic

#4960 Fix micro-tones assigned to strings in TabStaff

Status: Verified

Owner: Thomas Morley

Labels: Fixed_2_19_48 (6)

Needs:

Patch:

Type: Enhancement

Updated: 2016-11-04

Created: 2016-08-26

Creator: Thomas Morley

Private: No

Fix micro-tones assigned to strings in TabStaff

This is an oversight from issue 4643
Also extending input/regression/tablature-micro-tone.ly to cover this case.

http://codereview.appspot.com/308230043

1 Attachments

RegTestDiff.png

Discussion

Thomas Morley - 2016-08-26

Test-case:

\version "2.19.46" \layout { \context { \Score supportNonIntegerFret = ##t } } \new TabStaff { gih\6 }

Although it is printed correctly, needless warnings are printed.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2016-08-27

Description has changed:

Diff:

Attachments has changed:

Diff:

--- old +++ new @@ -0,0 +1 @@ +RegTestDiff.png (163.7 kB; image/png)

Needs: -->

Patch: new --> review
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2016-08-27

Passes make, make check and a full make doc.

Reg test diff attached

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Thomas Morley - 2016-08-27

Reg test diff as expected

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2016-08-30

Patch: review --> countdown
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2016-08-30

Patch on countdown for Sept 2nd

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2016-09-02

Patch: countdown --> push
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2016-09-02

Patch counted down - please push.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Thomas Morley - 2016-09-02

labels: --> Fixed_2_19_48

status: Started --> Fixed

Patch: push -->
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Thomas Morley - 2016-09-02

pushed to staging as:
commit 46a056a27c59a31ed53f20e27f8e2772dcbea0d0
Author: Thomas Morley thomasmorley65@gmail.com
Date: Sat Aug 27 00:32:30 2016 +0200

Issue 4960 Fix micro-tones assigned to strings in TabStaff This is an oversight from issue 4643 Also extending input/regression/tablature-micro-tone.ly to cover this case.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Federico Bruni - 2016-11-04

status: Fixed --> Verified
If you would like to refer to this comment somewhere else in this project, copy and paste the following link: