Re: filename() field should be const std::string* not std::string*

2020-05-25 Thread Martin Blais
On Mon, May 25, 2020 at 2:16 AM Akim Demaille wrote: > Hi Martin, > > > Le 24 mai 2020 à 16:56, Martin Blais a écrit : > > > > Dear grammar-generators, > > I'm setting up my %initial-action directive this morning (so much fun) > > I sense some true joy here :) If you see how to improve things,

Re: filename() field should be const std::string* not std::string*

2020-05-24 Thread Akim Demaille
Hi Martin, > Le 24 mai 2020 à 16:56, Martin Blais a écrit : > > Dear grammar-generators, > I'm setting up my %initial-action directive this morning (so much fun) I sense some true joy here :) If you see how to improve things, don't hesitate making suggestions. > and > I can't help notice that

filename() field should be const std::string* not std::string*

2020-05-24 Thread Martin Blais
Dear grammar-generators, I'm setting up my %initial-action directive this morning (so much fun) and I can't help notice that the C++ location() and position() constructors and initializers accept an "std::string*" and not a "const std::string*", despite not modifying their value anywhere. The docs