Re: Question about some fields in regex's re_pattern_buffer

[Paolo Bonzini]

On 08/14/2010 02:48 PM, Bruno Haible wrote:
> > >  re_set_registers is not documented; there's a note about this. I'd be
> > >  happy to write some documentation for that. There's also the technique
> > >  of re-using a registers data structure by NOT re_compile_fastmap, but
> > >  using the fastmap member of the struct directly. Finally, there's the
> > >  problem of the non-thread-safety of RE_NO_SUB which Paolo mentioned;
> > >  again, I can document that if we're not going to solve it immediately.
> I can't comment on this. Paolo?

The non-thread-safety isn't only for RE_NO_SUB, but for all syntax 
options.  RE_NO_SUB makes it worse only because it is _not_ a syntax 
option, but a setting to provide improved matching speed.

I think RE_NO_SUB should be documented, and a not on non-thread-safety 
of re_set_syntax_options should be added.

Paolo