[Lilypond-auto] [LilyIssues-auto] [testlilyissues:issues] #4917 Remove some set_location calls from parser

[Auto mailings of changes to Lily Issues]

• status: Fixed --> Verified

---

[issues:#4917] Remove some set_location calls from parser

Status: Verified
Labels: Fixed_2_19_46
Created: Mon Jul 04, 2016 08:58 AM UTC by David Kastrup
Last Updated: Mon Jul 11, 2016 07:33 AM UTC
Owner: David Kastrup

Remove some set_location calls from parser

Older versions of Bison (current is 3.0.4) had problems assigning
location data to rules with empty production, possibly related to the
definition of YYLLOCA_DEFAULT when N is zero.

This lead to several workarounds in the code base. A number of them
dropped through the floor in the course of refactoring without
apparent problem, and the original problem does not appear to be
reproducible with the current versions of Bison.

This removes the remaining instances. Should the original problem
reoccur at some point of time (or with some versions of Bison), it
would be noticeable as bad point-and-click messages and/or error
messages with bad location data.

http://codereview.appspot.com/298660043

---

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