<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>50292</bug_id>
          
          <creation_ts>2010-11-30 16:42:25 -0800</creation_ts>
          <short_desc>Port PCRE regex tests to JSC harness / ECMA syntax</short_desc>
          <delta_ts>2010-12-01 11:45:08 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Gavin Barraclough">barraclough</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>315400</commentid>
    <comment_count>0</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2010-11-30 16:42:25 -0800</bug_when>
    <thetext>LayoutTests/fast/regex currently contains a number of test cases ported from PCRE&apos;s test suites.
There are two key problems with these tests:
  * They use their own harness, and cannot be run from jsc without building all of WebKit.
  * They expect PCRE expression syntax, including extended syntax, backwards assertions, etc.

Rewrite all tests to comply to ECMA compatible syntax (ish - we support octal escapes), and use the normal &apos;shouldBe&apos; test style.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>315457</commentid>
    <comment_count>1</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2010-11-30 17:52:54 -0800</bug_when>
    <thetext>Fixed in r72990.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>315544</commentid>
    <comment_count>2</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2010-11-30 22:41:13 -0800</bug_when>
    <thetext>Gavin, did you check upstream PCRE for any interesting tests that were added since we imported the suite?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>315779</commentid>
    <comment_count>3</comment_count>
    <who name="Gavin Barraclough">barraclough</who>
    <bug_when>2010-12-01 11:45:08 -0800</bug_when>
    <thetext>No, I didn&apos;t have time to look at this.  My guess is that this would be an inefficient way to get high quality JS tests (since most of the PCRE tests seemed to focus on features not supported by ECMAScript).  But feel free to give this a shot, if you disagree &amp; think it may be worthwhile.</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>