Re: [PATCH 1/5] Remove parser/location global variable setup

[Trevor Daniels]

David Kastrup wrote Saturday, May 30, 2015 7:51 PM
 
> Oh, and regarding the %parser and %location names: GUILE itself uses
> names like that for several fluids.
> 
> At any rate: huh.  I propose getting rid of a 10-year old syntactic
> artifact and the only response are naming convention enquiries.  I mean:
> good to know that there are still developers alive. 

I should have responded, having spent quite some time yesterday 
browsing the patch set.  I tried to understand how it worked, but having 
little familiarity with C++, or even C for that matter, I decided the 
technique was too esoteric for me to understand, at least without a lot 
of background reading first.  Apart from commenting about my 
ignorance there seemed little to say at the time.

But of course the result is very welcome, removing a requirement which 
is irritating and mysterious when first encountered, and a nuisance ever
after.

> I still think it may have the potential of issue 2883 where its general
> availability lagged behind people becoming used to it.

Not really.  No one will miss the need to add parser location.  The change
just normalises the syntax to what one would expect anyway.  It will be
appreciated in the same way that good health is appreciated after
recovering from an illness.

Thanks for doing this, David.

Trevor