Re: parser.yy: remove `fraction' (issue 6399045)

[dak]

On 2012/07/15 09:14:00, dak wrote:
> message: On 2012/07/15 05:07:13, Keith wrote:
> > On Sat, 14 Jul 2012 19:14:41 -0700, <mailto:address@hidden> wrote:
> >
> > > Do we have evidence of people using 2/ 3 anywhere?
> >
> > I don't know of any scores with spaces alongside the / .

> As far as I know, it has never been documented or demonstrated.  The
previous
> support in the parser was more likely than not added because the lexer
> historically did not support fractions in all modes.  That was changed
about
> half a year ago without complaints.

Well, "half a year ago" is quite the exaggeration.  Looking for an
actual commit, I can only find
commit 0757febe2084fd8507334987b6f6b524888247c7
Author: David Kastrup <address@hidden>
Date:   Sun May 13 01:23:31 2012 +0200

    Issue 2530: Use fractions rather than Scheme pairs for
\scaleDuration, timeS

I thought I remembered also earlier work on this (and I meddled a _lot_
with the parser regarding fractions), but apparently I had not
previously messed with the _lexer;s_ view of fractions.

So it has just been version 2.15.39 when 2/4 turned into a LilyPond
fraction rather than a numeric expression even in INITIAL state.

http://codereview.appspot.com/6399045/