-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
state variables are no longer working as expected in regex code blocks (/^ <?{ say $++ }> /) #6174
Comments
From @AlexDanielCode: Result (2015.12,2016.10): Result (2016.11,HEAD(099512b)): Bisectable points to rakudo/rakudo@7a456ff |
From @jnthnOn Tue, 04 Apr 2017 18:43:14 -0700, alex.jakimenko@gmail.com wrote:
A state variable works per closure clone, but we must re-clone the inner blocks in a regex to avoid threads stomping on each other's state. So this only "worked" previously thanks to a code-gen bug, which I fixed in the referenced commit. |
The RT System itself - Status changed from 'new' to 'open' |
From @AlexDanielWhich basically means that state variables are useless in this case. Is there any potential possibility to detect this and print some kind of a warning? On 2017-04-07 05:34:38, jnthn@jnthn.net wrote:
|
From joshuamy@gmail.comAn array of tokens used to be able to match like this: my token foo { 'foo' }; OUTPUT: 「foo bar baz」 This no longer work since commit 7a456ff80183a6e26dc91d811d992112c68ddb6d You can simulate how it should work by providing your own state variable state $i = 0; |
From @AlexDanielI'm fairly sure that the ship for this has sailed, we've had this “issue” for a year now. I guess during that year everyone has adapted their code to the new behavior. So if you're reading this comment and you think that the ticket should be closed, please just mark it as 「testneeded」 so that we have the new behavior cast in stone. And that'll be it. On 2017-08-06 08:32:55, joshuamy@gmail.com wrote:
|
Migrated from rt.perl.org#131102 (status was 'open')
Searchable as RT131102$
The text was updated successfully, but these errors were encountered: