Steve Fink (via RT) wrote:
- The .local directive requires a type. I fixed the documentation. - The lexer allows an optional dot in front of a parrot op or identifier, but then seems to assume that the dot is not there. So I took it off. - A few trivial things: parenthesized a macro arg for safety, fixed a typo, added a missing va_end.
Yep.
- Implemented one idea of namespaces.
[ ...]
so that I can just use 'rx_pos' within my (possibly nested) namespace.
So that's what this patch implements. A .local <type> <varname>
directive now creates a variable named <current-namespace>::<varname>
(or just <varname> at the top level),
Looks ok too. If no one hollers, I'll apply it.
I don't know, if your are speaking of the current rx.c or a new regex implementation, but the current "stateless" approach has some drawbacks: it leaks memory (bitmap) and isn't reentrant (intstack).
There should probably be some regex PMC that can store the state of a regex.
leo
