alexSetUserState/alexGetUserState with monadUserState-bytestring wrapper

[zoeyfyi]

Is there any reason these are not enabled for the monadUserState-bytestring wrapper?

alex/data/AlexWrappers.hs

Line 261 in 24ab083

#if !defined(ALEX_MONAD_BYTESTRING) && defined(ALEX_MONAD_USER_STATE)

[andreasabel]

I suggest you try to find the author (e.g. via git blame) and then ask the author...

[emekoi]

Tracing the change back to 13fd23e and 4de7cd8, it seems that this was a mistake.

[andreasabel]

Tracing the change back to 13fd23e and 4de7cd8, it seems that this was a mistake.

@emekoi : Is there a released version of alex that supports alex{G,S}etUserState in the monadUserState-bytestring wrapper?
According to my experiments,

• alex-3.1.7 (Jan 2016) does not know about that wrapper
• alex-3.2.0 (Oct 2016) knows about the wrapper but does not support alex{G,S}etUserState

So maybe this isn't a regression, but a feature that never worked.

[emekoi]

I probably should've checked, but I think you're right that this never worked. In a4f43f8 (just before 13fd23e) monadUserState-bytestring also did not support alex{G,S}etUserState. I probably just assumed alex did at some point because I was following this tutorial by Serokell, and they use alex{G,S}etUserState with monadUserState-bytestring.

Also could you clarify what you meant by "alex-3.1.7 (Jan 2016) does not know about that wrapper"? Looking at Github, alex{G,S}etUserState were introduced in #23 which made it into alex-3.1.1, and monadUserState-bytestring was introduced in alex-2.3.1 (57d9d6d).

[andreasabel]

Also could you clarify what you meant by "alex-3.1.7 (Jan 2016) does not know about that wrapper"?

Oh, I got a parse error for the test, so I assumed it was because of the %wrapper directive. I should check again...

[andreasabel]

I checked 3.1.4; it also does not know about alex{S,G}etUserState. Older versions of alex can't be built on my mac.