<?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>72288</bug_id>
          
          <creation_ts>2011-11-14 10:50:06 -0800</creation_ts>
          <short_desc>Towards 8 Bit Strings: Templatize YARR Parser</short_desc>
          <delta_ts>2011-11-14 11:25:54 -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>All</rep_platform>
          <op_sys>All</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>
          
          <blocked>71337</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Michael Saboff">msaboff</reporter>
          <assigned_to name="Michael Saboff">msaboff</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>502082</commentid>
    <comment_count>0</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-11-14 10:50:06 -0800</bug_when>
    <thetext>The current YARR parser calls StringImpl::characters().  Converting the parser to a template will allow the code to call characters8() or characters16() as appropriate.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>502093</commentid>
    <comment_count>1</comment_count>
      <attachid>114987</attachid>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-11-14 11:01:08 -0800</bug_when>
    <thetext>Created attachment 114987
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>502096</commentid>
    <comment_count>2</comment_count>
      <attachid>114987</attachid>
    <who name="Oliver Hunt">oliver</who>
    <bug_when>2011-11-14 11:04:13 -0800</bug_when>
    <thetext>Comment on attachment 114987
Patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>502124</commentid>
    <comment_count>3</comment_count>
    <who name="Michael Saboff">msaboff</who>
    <bug_when>2011-11-14 11:25:54 -0800</bug_when>
    <thetext>Committed r100167: &lt;http://trac.webkit.org/changeset/100167&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>114987</attachid>
            <date>2011-11-14 11:01:08 -0800</date>
            <delta_ts>2011-11-14 11:04:12 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>72288.patch</filename>
            <type>text/plain</type>
            <size>2325</size>
            <attacher name="Michael Saboff">msaboff</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTAwMTYyKQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBA
CisyMDExLTExLTE0ICBNaWNoYWVsIFNhYm9mZiAgPG1zYWJvZmZAYXBwbGUuY29tPgorCisgICAg
ICAgIFRvd2FyZHMgOCBCaXQgU3RyaW5nczogVGVtcGxhdGl6ZSBZQVJSIFBhcnNlcgorICAgICAg
ICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NzIyODgKKworICAgICAg
ICBDaGFuZ2VkIFlhcnI6OlBhcnNlciB0byBiZSBhIHRlbXBsYXRlIGJhc2VkIG9uIGNoYXJhY3Rl
ciB0eXBlLgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAg
ICogeWFyci9ZYXJyUGFyc2VyLmg6CisgICAgICAgIChKU0M6OllhcnI6OlBhcnNlcjo6UGFyc2Vy
KToKKyAgICAgICAgKEpTQzo6WWFycjo6cGFyc2UpOgorCiAyMDExLTExLTExICBKZXIgTm9ibGUg
IDxqZXIubm9ibGVAYXBwbGUuY29tPgogCiAgICAgICAgIEltcGxlbWVudCBNZWRpYUNvbnRyb2xs
ZXIuCkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUveWFyci9ZYXJyUGFyc2VyLmgKPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL3lhcnIvWWFyclBhcnNlci5oCShyZXZpc2lv
biA5OTg2MSkKKysrIFNvdXJjZS9KYXZhU2NyaXB0Q29yZS95YXJyL1lhcnJQYXJzZXIuaAkod29y
a2luZyBjb3B5KQpAQCAtNDMsNyArNDMsNyBAQCBlbnVtIEJ1aWx0SW5DaGFyYWN0ZXJDbGFzc0lE
IHsKIH07CiAKIC8vIFRoZSBQYXJzZXIgY2xhc3Mgc2hvdWxkIG5vdCBiZSB1c2VkIGRpcmVjdGx5
IC0gb25seSB2aWEgdGhlIFlhcnI6OnBhcnNlKCkgbWV0aG9kLgotdGVtcGxhdGU8Y2xhc3MgRGVs
ZWdhdGU+Cit0ZW1wbGF0ZTxjbGFzcyBEZWxlZ2F0ZSwgdHlwZW5hbWUgQ2hhclR5cGU+CiBjbGFz
cyBQYXJzZXIgewogcHJpdmF0ZToKICAgICB0ZW1wbGF0ZTxjbGFzcyBGcmllbmREZWxlZ2F0ZT4K
QEAgLTIzMSwxMyArMjMxLDEzIEBAIHByaXZhdGU6CiAgICAgICAgIDogbV9kZWxlZ2F0ZShkZWxl
Z2F0ZSkKICAgICAgICAgLCBtX2JhY2tSZWZlcmVuY2VMaW1pdChiYWNrUmVmZXJlbmNlTGltaXQp
CiAgICAgICAgICwgbV9lcnIoTm9FcnJvcikKLSAgICAgICAgLCBtX2RhdGEocGF0dGVybi5jaGFy
YWN0ZXJzMTYoKSkKKyAgICAgICAgLCBtX2RhdGEocGF0dGVybi5nZXRDaGFyYWN0ZXJzPENoYXJU
eXBlPigpKQogICAgICAgICAsIG1fc2l6ZShwYXR0ZXJuLmxlbmd0aCgpKQogICAgICAgICAsIG1f
aW5kZXgoMCkKICAgICAgICAgLCBtX3BhcmVudGhlc2VzTmVzdGluZ0RlcHRoKDApCiAgICAgewog
ICAgIH0KLSAgICAKKwogICAgIC8qCiAgICAgICogcGFyc2VFc2NhcGUoKToKICAgICAgKgpAQCAt
NzkzLDcgKzc5Myw3IEBAIHByaXZhdGU6CiAgICAgRGVsZWdhdGUmIG1fZGVsZWdhdGU7CiAgICAg
dW5zaWduZWQgbV9iYWNrUmVmZXJlbmNlTGltaXQ7CiAgICAgRXJyb3JDb2RlIG1fZXJyOwotICAg
IGNvbnN0IFVDaGFyKiBtX2RhdGE7CisgICAgY29uc3QgQ2hhclR5cGUqIG1fZGF0YTsKICAgICB1
bnNpZ25lZCBtX3NpemU7CiAgICAgdW5zaWduZWQgbV9pbmRleDsKICAgICB1bnNpZ25lZCBtX3Bh
cmVudGhlc2VzTmVzdGluZ0RlcHRoOwpAQCAtODY0LDcgKzg2NCw5IEBAIHByaXZhdGU6CiB0ZW1w
bGF0ZTxjbGFzcyBEZWxlZ2F0ZT4KIGNvbnN0IGNoYXIqIHBhcnNlKERlbGVnYXRlJiBkZWxlZ2F0
ZSwgY29uc3QgVVN0cmluZyYgcGF0dGVybiwgdW5zaWduZWQgYmFja1JlZmVyZW5jZUxpbWl0ID0g
cXVhbnRpZnlJbmZpbml0ZSkKIHsKLSAgICByZXR1cm4gUGFyc2VyPERlbGVnYXRlPihkZWxlZ2F0
ZSwgcGF0dGVybiwgYmFja1JlZmVyZW5jZUxpbWl0KS5wYXJzZSgpOworICAgIGlmIChwYXR0ZXJu
LmlzOEJpdCgpKQorICAgICAgICByZXR1cm4gUGFyc2VyPERlbGVnYXRlLCBMQ2hhcj4oZGVsZWdh
dGUsIHBhdHRlcm4sIGJhY2tSZWZlcmVuY2VMaW1pdCkucGFyc2UoKTsKKyAgICByZXR1cm4gUGFy
c2VyPERlbGVnYXRlLCBVQ2hhcj4oZGVsZWdhdGUsIHBhdHRlcm4sIGJhY2tSZWZlcmVuY2VMaW1p
dCkucGFyc2UoKTsKIH0KIAogfSB9IC8vIG5hbWVzcGFjZSBKU0M6OllhcnIK
</data>
<flag name="review"
          id="113766"
          type_id="1"
          status="+"
          setter="oliver"
    />
          </attachment>
      

    </bug>

</bugzilla>