Skip to content
  • darin@apple.com's avatar
    JavaScriptCore: · 9be07b6f
    darin@apple.com authored
            Reviewed by Maciej.
    
            - fix http://bugs.webkit.org/show_bug.cgi?id=16459
              REGRESSION: assertion failure with regexp with \B in a case-ignoring character range
              <rdar://problem/5646361>
    
            The problem was that \B was not handled properly in character classes.
    
            Test: fast/js/regexp-overflow.html
    
            * pcre/pcre_compile.cpp:
            (check_escape): Added handling of ESC_b and ESC_B in character classes here.
            Allows us to get rid of the handling of \b in character classes from all the
            call sites that handle it separately and to handle \B properly as well.
            (compileBranch): Remove the ESC_b handling, since it's not needed any more.
            (calculateCompiledPatternLengthAndFlags): Ditto.
    
    LayoutTests:
    
            Reviewed by Maciej.
    
            - test for http://bugs.webkit.org/show_bug.cgi?id=16459
              REGRESSION: assertion failure with regexp with \B in a case-ignoring character range
              <rdar://problem/5646361>
    
            * fast/js/regexp-overflow-expected.txt: Updated.
            * fast/js/resources/regexp-overflow.js: Added test cases.
    
    
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@28785 268f45cc-cd09-0410-ab3c-d52691b4dbfc
    9be07b6f