[pcre-dev] [Bug 1493] pcre_study does not include VT in star…

Top Page
Delete this message
Author: Zoltan Herczeg
Date:  
To: pcre-dev
Subject: [pcre-dev] [Bug 1493] pcre_study does not include VT in starting chars for /\s/
------- You are receiving this mail because: -------
You are on the CC list for the bug.

http://bugs.exim.org/show_bug.cgi?id=1493

Zoltan Herczeg <hzmester@???> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hzmester@???





--- Comment #1 from Zoltan Herczeg <hzmester@???> 2014-06-17 17:03:54 ---
Hi Justin,

thanks for reporting these various issues. This is really useful for the
project.

This one is interesting. SVN annotate says:

  1033       ph10       /* The cbit_space table has vertical tab as whitespace;
we have to not
  1033       ph10       set it from the table. Luckily, the code value is the
same (0x0b) in
  1033       ph10       ASCII and EBCDIC, so we can just adjust the appropriate
bit. */
    91      nigel
    77      nigel       case OP_WHITESPACE:
   539       ph10       c = start_bits[1];    /* Save in case it was already
set */
   539       ph10       set_type_bits(start_bits, cbit_space, table_limit, cd);
   539       ph10       start_bits[1] = (start_bits[1] & ~0x08) | c;
    77      nigel       try_next = FALSE;
    77      nigel       break;


PCRE woprks this way since r539. Philip even added a comment in r1033. I
suspect we could simply remove this special case.


--
Configure bugmail: http://bugs.exim.org/userprefs.cgi?tab=email