<?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>22107</bug_id>
          
          <creation_ts>2008-11-06 10:56:56 -0800</creation_ts>
          <short_desc>Fix compilation of WTF::AVLTree</short_desc>
          <delta_ts>2008-11-06 11:31:17 -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>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="Simon Hausmann">hausmann</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>97842</commentid>
    <comment_count>0</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-11-06 10:56:56 -0800</bug_when>
    <thetext>Some functions in AVLTree call get_lt and get_gt with a second argument that does not exist.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97843</commentid>
    <comment_count>1</comment_count>
      <attachid>24948</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-11-06 10:57:24 -0800</bug_when>
    <thetext>Created attachment 24948
Patch to fix compilation</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97846</commentid>
    <comment_count>2</comment_count>
      <attachid>24948</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2008-11-06 11:19:42 -0800</bug_when>
    <thetext>Comment on attachment 24948
Patch to fix compilation

&gt; +        * wtf/AVLTree.h:
&gt; +        (JSC::::remove):
&gt; +        (JSC::::subst):

This ChangeLog entry looks strange. I know it was produced by prepare-ChangeLog, but we shouldn&apos;t check in strange log entries just because the script made them.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97850</commentid>
    <comment_count>3</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-11-06 11:30:57 -0800</bug_when>
    <thetext>Thanks, landed with corrected changelog in r38190</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>97851</commentid>
    <comment_count>4</comment_count>
      <attachid>24948</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-11-06 11:31:17 -0800</bug_when>
    <thetext>Comment on attachment 24948
Patch to fix compilation

(clearing review after landing)</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>24948</attachid>
            <date>2008-11-06 10:57:24 -0800</date>
            <delta_ts>2008-11-06 11:31:17 -0800</delta_ts>
            <desc>Patch to fix compilation</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>2427</size>
            <attacher name="Simon Hausmann">hausmann</attacher>
            
              <data encoding="base64">Y29tbWl0IGM5YjdhOTA2N2RkMDhiMzg0NmI0YjlhNGE0NmYyYzk0ODAwOGZjZTcKQXV0aG9yOiBM
YXN6bG8gR29tYm9zIDxsYXN6bG8uMS5nb21ib3NAbm9raWEuY29tPgpEYXRlOiAgIFRodSBPY3Qg
MjMgMjE6NDE6NDcgMjAwOCArMDMwMAoKICAgIDAwOC0xMS0wNiAgTGFzemxvIEdvbWJvcyAgPGxh
c3psby4xLmdvbWJvc0Bub2tpYS5jb20+CiAgICAKICAgICAgICAgICAgUmV2aWV3ZWQgYnkgU2lt
b24gSGF1c21hbm4uCiAgICAKICAgICAgICAgICAgQnVnIHVuY292ZXJlZCBkdXJpbmcgUlZDVCBw
b3J0IGluIGZ1bmN0aW9ucyBub3QgdXNlZC4gZ2V0X2x0KCkgYW5kCiAgICAgICAgICAgIGdldF9n
dCgpIHRha2VzIG9ubHkgb25lIGFyZ3VtZW50IC0gcmVtb3ZlIHNlY29uZCBhcmd1bWVudCB3aGVy
ZQogICAgICAgICAgICBhcHBsaWNhYmxlLgoKZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0No
YW5nZUxvZyBiL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwppbmRleCBjMzIxNDk3Li4xYTBjZjU0
IDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKKysrIGIvSmF2YVNjcmlwdENv
cmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUgQEAKKzIwMDgtMTEtMDYgIExhc3psbyBHb21ib3Mg
IDxsYXN6bG8uMS5nb21ib3NAbm9raWEuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IFNpbW9u
IEhhdXNtYW5uLgorCisgICAgICAgIEJ1ZyB1bmNvdmVyZWQgZHVyaW5nIFJWQ1QgcG9ydCBpbiBm
dW5jdGlvbnMgbm90IHVzZWQuIGdldF9sdCgpIGFuZAorICAgICAgICBnZXRfZ3QoKSB0YWtlcyBv
bmx5IG9uZSBhcmd1bWVudCAtIHJlbW92ZSBzZWNvbmQgYXJndW1lbnQgd2hlcmUKKyAgICAgICAg
YXBwbGljYWJsZS4KKworICAgICAgICAqIHd0Zi9BVkxUcmVlLmg6CisgICAgICAgIChKU0M6Ojo6
cmVtb3ZlKToKKyAgICAgICAgKEpTQzo6OjpzdWJzdCk6CisKIDIwMDgtMTEtMDYgIEFscCBUb2tl
ciAgPGFscEBudWFudGkuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IENhbWVyb24gWndhcmlj
aC4KZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL3d0Zi9BVkxUcmVlLmggYi9KYXZhU2NyaXB0
Q29yZS93dGYvQVZMVHJlZS5oCmluZGV4IGNkMTUxMWYuLjk3ZjI4YWIgMTAwNjQ0Ci0tLSBhL0ph
dmFTY3JpcHRDb3JlL3d0Zi9BVkxUcmVlLmgKKysrIGIvSmF2YVNjcmlwdENvcmUvd3RmL0FWTFRy
ZWUuaApAQCAtODIyLDcgKzgyMiw3IEBAIEFWTFRyZWU8QWJzdHJhY3RvciwgbWF4RGVwdGgsIEJT
ZXQ+OjpyZW1vdmUoa2V5IGspCiAgICAgICAgICAgICBjbXBfc2hvcnRlbmVkX3N1Yl93aXRoX3Bh
dGggPSBjbXA7CiAKICAgICAgICAgLy8gR2V0IHRoZSBoYW5kbGUgb2YgdGhlIG9wcG9zaXRlIGNo
aWxkLCB3aGljaCBtYXkgbm90IGJlIG51bGwuCi0gICAgICAgIGNoaWxkID0gY21wID4gMCA/IGdl
dF9sdChoLCBmYWxzZSkgOiBnZXRfZ3QoaCwgZmFsc2UpOworICAgICAgICBjaGlsZCA9IGNtcCA+
IDAgPyBnZXRfbHQoaCkgOiBnZXRfZ3QoaCk7CiAgICAgfQogCiAgICAgaWYgKHBhcmVudCA9PSBu
dWxsKCkpCkBAIC04NDEsOCArODQxLDggQEAgQVZMVHJlZTxBYnN0cmFjdG9yLCBtYXhEZXB0aCwg
QlNldD46OnJlbW92ZShrZXkgaykKIAogICAgIGlmIChoICE9IHJtKSB7CiAgICAgICAgIC8vIFBv
a2UgaW4gdGhlIHJlcGxhY2VtZW50IGZvciB0aGUgbm9kZSB0byBiZSByZW1vdmVkLgotICAgICAg
ICBzZXRfbHQoaCwgZ2V0X2x0KHJtLCBmYWxzZSkpOwotICAgICAgICBzZXRfZ3QoaCwgZ2V0X2d0
KHJtLCBmYWxzZSkpOworICAgICAgICBzZXRfbHQoaCwgZ2V0X2x0KHJtKSk7CisgICAgICAgIHNl
dF9ndChoLCBnZXRfZ3Qocm0pKTsKICAgICAgICAgc2V0X2JmKGgsIGdldF9iZihybSkpOwogICAg
ICAgICBpZiAocGFyZW50X3JtID09IG51bGwoKSkKICAgICAgICAgICAgIGFicy5yb290ID0gaDsK
QEAgLTkzNCw4ICs5MzQsOCBAQCBBVkxUcmVlPEFic3RyYWN0b3IsIG1heERlcHRoLCBCU2V0Pjo6
c3Vic3QoaGFuZGxlIG5ld19ub2RlKQogICAgIH0KIAogICAgIC8qIENvcHkgdHJlZSBob3VzZWtl
ZXBpbmcgZmllbGRzIGZyb20gbm9kZSBpbiB0cmVlIHRvIG5ldyBub2RlLiAqLwotICAgIHNldF9s
dChuZXdfbm9kZSwgZ2V0X2x0KGgsIGZhbHNlKSk7Ci0gICAgc2V0X2d0KG5ld19ub2RlLCBnZXRf
Z3QoaCwgZmFsc2UpKTsKKyAgICBzZXRfbHQobmV3X25vZGUsIGdldF9sdChoKSk7CisgICAgc2V0
X2d0KG5ld19ub2RlLCBnZXRfZ3QoaCkpOwogICAgIHNldF9iZihuZXdfbm9kZSwgZ2V0X2JmKGgp
KTsKIAogICAgIGlmIChwYXJlbnQgPT0gbnVsbCgpKQpAQCAtOTUyLDcgKzk1Miw2IEBAIEFWTFRy
ZWU8QWJzdHJhY3RvciwgbWF4RGVwdGgsIEJTZXQ+OjpzdWJzdChoYW5kbGUgbmV3X25vZGUpCiAg
ICAgcmV0dXJuIGg7CiB9CiAKLQogfQogCiAjZW5kaWYK
</data>

          </attachment>
      

    </bug>

</bugzilla>