Re: fingering-engraver.cc: don't set Fingering.pitch (issue 347930043 by address@hidden)

[dak]

Reviewers: lemzwerg,

Message:
On 2018/08/03 03:24:40, lemzwerg wrote:
> LGTM.  I suggest that you do such trivial clean-ups directly in the
git
> repository, not wasting your time with setting up an issue.

If Han-Wen did not see fit to remove the line but instead went to the
trouble of adding a "WTF?" comment, I think removing without an
opportunity for people to check would be optimistic.  Also I leave a
record that way.

Description:
fingering-engraver.cc: don't set Fingering.pitch

This grob property does not actually exist.

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

Affected files (+0, -9 lines):
  M lily/fingering-engraver.cc


Index: lily/fingering-engraver.cc
diff --git a/lily/fingering-engraver.cc b/lily/fingering-engraver.cc
index  
4f40bff00e7a1048aed2b02fdc63e088ad14fa29..7d77f776f59707ccf0d8c41981f7a505438772b7  
100644
--- a/lily/fingering-engraver.cc
+++ b/lily/fingering-engraver.cc
@@ -94,15 +94,6 @@ Fingering_engraver::make_script (Direction d,  
Stream_event *r, int i)
 {
   Item *fingering = make_item ("Fingering", r->self_scm ());

-  /*
-    Huh, what's this for? --hwn.
-
-    junkme.
-  */
-  SCM pitch = r->get_property ("pitch");
-  if (unsmob<Pitch> (pitch))
-    fingering->set_property ("pitch", pitch);
-
   /*
     We can't fold these definitions into define-grobs since
     fingerings for chords need different settings.