location tracking for operators about lexy HOT 2 CLOSED

1. The constructor of the tag is passed an iterator from which a location can be computed. My current workaround is to store the iterator in a std::any and do the mapping later. However, I am not a fan of using this type. I add a patch with a proposal to pass the state to the tag constructor, below.

That seems like a sensible addition. If you want, you can turn the patch into a PR, by also adding a test and documentation. (And incorporate the fix for 2.)

2. Somehow the iterator passed to the tag is off by one. See the example below reproducing the issue.

Fixed.

3. I don't like to use lexy::bind and rather use lexy::callback with lambdas. Would it be possible to have a convenience version that also receives the parse state?

I have added lexy::callback_with_state for that purpose.

from lexy.

Thanks, for the two additions! I'll open a PR, too. It can take 2 to 3 weeks though.

from lexy.