<?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>54263</bug_id>
          
          <creation_ts>2011-02-10 18:44:58 -0800</creation_ts>
          <short_desc>add a findRunOfZeros() method to wtf/Bitmap.h</short_desc>
          <delta_ts>2011-02-11 13:15:42 -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>Web Template Framework</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords>Qt</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          <blocked>51128</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Siddharth Mathur">s.mathur</reporter>
          <assigned_to name="Siddharth Mathur">s.mathur</assigned_to>
          <cc>barraclough</cc>
    
    <cc>ggaren</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>nlawrence</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>349557</commentid>
    <comment_count>0</comment_count>
    <who name="Siddharth Mathur">s.mathur</who>
    <bug_when>2011-02-10 18:44:58 -0800</bug_when>
    <thetext>Additional method in wtf/Bitmap.h that locates (the start index of) a run of N zeros in the bitmap. 
This will help me complete Bug 51128 without rolling my own bit map implementation.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>349565</commentid>
    <comment_count>1</comment_count>
      <attachid>82089</attachid>
    <who name="Siddharth Mathur">s.mathur</who>
    <bug_when>2011-02-10 18:57:29 -0800</bug_when>
    <thetext>Created attachment 82089
Patch for view

Almost brute force search through the bit array to find a run of zero bit. 

(The return type of int64_t is a bit awkward, but I wanted to return a -1 in case of &quot;not found&quot; while also be able to represent 2^32)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>349951</commentid>
    <comment_count>2</comment_count>
      <attachid>82089</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2011-02-11 13:04:12 -0800</bug_when>
    <thetext>Comment on attachment 82089
Patch for view

We don’t normally take patches that add in functions that aren’t used. For one ting, such functions can’t be tested.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>349963</commentid>
    <comment_count>3</comment_count>
    <who name="Siddharth Mathur">s.mathur</who>
    <bug_when>2011-02-11 13:15:42 -0800</bug_when>
    <thetext>OK, I will make it part of Bug 51128. Thanks.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>82089</attachid>
            <date>2011-02-10 18:57:29 -0800</date>
            <delta_ts>2011-02-11 13:04:12 -0800</delta_ts>
            <desc>Patch for view</desc>
            <filename>54263_v1.patch</filename>
            <type>text/plain</type>
            <size>1756</size>
            <attacher name="Siddharth Mathur">s.mathur</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gNzgyOTgpCisrKyBTb3VyY2Uv
SmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTMgQEAK
KzIwMTEtMDItMTAgIFNpZGRoYXJ0aCBNYXRodXIgIDxzaWRkaGFydGgubWF0aHVyQG5va2lhLmNv
bT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBhZGQg
YSBmaW5kUnVuT2ZaZXJvcygpIG1ldGhvZCB0byB3dGYvQml0bWFwLmgKKyAgICAgICAgaHR0cDov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NTQyNjMgCisKKyAgICAgICAgKiB3dGYv
Qml0bWFwLmg6CisgICAgICAgIChXVEY6Ojo6ZmluZFJ1bk9mWmVyb3MpOiBsb2NhdGUgYSB3aW5k
b3dzIG9mIE4gdW5zZXQgYml0cworCiAyMDExLTAyLTEwICBNaWNoYWVsIFNhYm9mZiAgPG1zYWJv
ZmZAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IEdlb2ZmcmV5IEdhcmVuLgpJbmRl
eDogU291cmNlL0phdmFTY3JpcHRDb3JlL3d0Zi9CaXRtYXAuaAo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3Vy
Y2UvSmF2YVNjcmlwdENvcmUvd3RmL0JpdG1hcC5oCShyZXZpc2lvbiA3NTM3NSkKKysrIFNvdXJj
ZS9KYXZhU2NyaXB0Q29yZS93dGYvQml0bWFwLmgJKHdvcmtpbmcgY29weSkKQEAgLTM5LDYgKzM5
LDcgQEAgcHVibGljOgogICAgIHZvaWQgY2xlYXIoc2l6ZV90KTsKICAgICB2b2lkIGNsZWFyQWxs
KCk7CiAgICAgdm9pZCBhZHZhbmNlVG9OZXh0RnJlZUJpdChzaXplX3QmKSBjb25zdDsKKyAgICBp
bnQ2NF90IGZpbmRSdW5PZlplcm9zKHNpemVfdCkgY29uc3Q7CiAgICAgc2l6ZV90IGNvdW50KHNp
emVfdCA9IDApIGNvbnN0OwogICAgIHNpemVfdCBpc0VtcHR5KCkgY29uc3Q7CiAgICAgc2l6ZV90
IGlzRnVsbCgpIGNvbnN0OwpAQCAtOTcsNiArOTgsMjYgQEAgaW5saW5lIHZvaWQgQml0bWFwPHNp
emU+OjphZHZhbmNlVG9OZXh0RgogfQogCiB0ZW1wbGF0ZTxzaXplX3Qgc2l6ZT4KK2lubGluZSBp
bnQ2NF90IEJpdG1hcDxzaXplPjo6ZmluZFJ1bk9mWmVyb3Moc2l6ZV90IHJ1bkxlbmd0aCkgY29u
c3QKK3sKKyAgICBpZiAoIXJ1bkxlbmd0aCkgCisgICAgICAgICAgICBydW5MZW5ndGggPSAxOyAK
KyAgICAgCisgICAgZm9yIChzaXplX3QgaSA9IDA7IGkgPD0gKHNpemUgLSBydW5MZW5ndGgpIDsg
aSsrKSB7CisgICAgICAgIGJvb2wgZm91bmQgPSB0cnVlOyAKKyAgICAgICAgZm9yIChzaXplX3Qg
aiA9IGk7IGogPD0gKGkgKyBydW5MZW5ndGggLSAxKSA7IGorKykgeyAKKyAgICAgICAgICAgIGlm
IChnZXQoaikpIHsKKyAgICAgICAgICAgICAgICBmb3VuZCA9IGZhbHNlOyAKKyAgICAgICAgICAg
ICAgICBicmVhazsKKyAgICAgICAgICAgIH0KKyAgICAgICAgfQorICAgICAgICBpZiAoZm91bmQp
ICAKKyAgICAgICAgICAgIHJldHVybiBpOyAKKyAgICB9CisgICAgcmV0dXJuIC0xOworfQorCit0
ZW1wbGF0ZTxzaXplX3Qgc2l6ZT4KIGlubGluZSBzaXplX3QgQml0bWFwPHNpemU+Ojpjb3VudChz
aXplX3Qgc3RhcnQpIGNvbnN0CiB7CiAgICAgc2l6ZV90IHJlc3VsdCA9IDA7Cg==
</data>
<flag name="review"
          id="73723"
          type_id="1"
          status="-"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>