<?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>78703</bug_id>
          
          <creation_ts>2012-02-15 05:31:26 -0800</creation_ts>
          <short_desc>Add script to determine which functions are affected by a patch</short_desc>
          <delta_ts>2012-11-22 03:49:16 -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>Tools / Tests</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>WONTFIX</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>78707</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ossy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>557183</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-02-15 05:31:26 -0800</bug_when>
    <thetext>It is needed for selective testing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>557237</commentid>
    <comment_count>1</comment_count>
      <attachid>127177</attachid>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-15 06:56:23 -0800</bug_when>
    <thetext>Created attachment 127177
Patch

Determines the functions names with the help of prepare-ChangeLog.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>557342</commentid>
    <comment_count>2</comment_count>
      <attachid>127177</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2012-02-15 10:00:48 -0800</bug_when>
    <thetext>Comment on attachment 127177
Patch

The idea of this script seems fine. The name “generate functions” seems totally wrong though, since the script that does not generate functions. Maybe a name like “list modified functions”?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>558013</commentid>
    <comment_count>3</comment_count>
      <attachid>127335</attachid>
    <who name="Nandor Huszka">hnandor</who>
    <bug_when>2012-02-16 02:33:21 -0800</bug_when>
    <thetext>Created attachment 127335
Patch v2

(In reply to comment #10)
Thank you for the comment, I have renamed it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>564447</commentid>
    <comment_count>4</comment_count>
      <attachid>127335</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2012-02-24 10:41:51 -0800</bug_when>
    <thetext>Comment on attachment 127335
Patch v2

Why do we need a separate script for this?  If you&apos;re planning to use this in webkitpy, we already have a function called changed_files which does this work.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>773812</commentid>
    <comment_count>5</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2012-11-22 03:49:16 -0800</bug_when>
    <thetext>You&apos;re right, we don&apos;t need separated script for this.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>127177</attachid>
            <date>2012-02-15 06:56:23 -0800</date>
            <delta_ts>2012-02-16 02:33:21 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>generatefunctions.diff</filename>
            <type>text/plain</type>
            <size>2345</size>
            <attacher name="Nandor Huszka">hnandor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBhZTdm
MGU0Li5mY2RlZGVhIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTItMDItMTUgIE7DoW5kb3IgSHVzemthICA8aHVz
emthLm5hbmRvckBzdHVkLnUtc3plZ2VkLmh1PgorCisgICAgICAgIEFkZCBzY3JpcHQgdG8gZGV0
ZXJtaW5lIHdoaWNoIGZ1bmN0aW9ucyBhcmUgYWZmZWN0ZWQgYnkgYSBwYXRjaGFuYWx5c2lzdGFz
aworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Nzg3MDMK
KworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBVc2VzIHRo
ZSBwcmVwYXJlLUNoYW5nZUxvZydzIC0tbm8td3JpdGUgb3B0aW9uIHRvIGRldGVybWluZSB0aGUg
bW9kaWZpZWQgZnVuY3Rpb25zLgorCisgICAgICAgICogU2NyaXB0cy9nZW5lcmF0ZS1mdW5jdGlv
bnM6IEFkZGVkLgorCiAyMDEyLTAyLTE0ICBSYW15YSBDaGFuZHJhc2VrYXJhbiAgPGNyYW15YUBn
b29nbGUuY29tPgogCiAgICAgICAgIExhc3QgY2hhcmFjdGVyIGRpc3BsYXkgZm9yIHBhc3N3b3Jk
cyBpbiBBbmRyb2lkLgpkaWZmIC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9nZW5lcmF0ZS1mdW5jdGlv
bnMgYi9Ub29scy9TY3JpcHRzL2dlbmVyYXRlLWZ1bmN0aW9ucwpuZXcgZmlsZSBtb2RlIDEwMDc1
NQppbmRleCAwMDAwMDAwLi4yNzY1YjY2Ci0tLSAvZGV2L251bGwKKysrIGIvVG9vbHMvU2NyaXB0
cy9nZW5lcmF0ZS1mdW5jdGlvbnMKQEAgLTAsMCArMSwyNiBAQAorIyBDb3B5cmlnaHQgKEMpIDIw
MTIgVW5pdmVyc2l0eSBvZiBTemVnZWQuIEFsbCByaWdodHMgcmVzZXJ2ZWQuCisjCisjIFJlZGlz
dHJpYnV0aW9uIGFuZCB1c2UgaW4gc291cmNlIGFuZCBiaW5hcnkgZm9ybXMsIHdpdGggb3Igd2l0
aG91dAorIyBtb2RpZmljYXRpb24sIGFyZSBwZXJtaXR0ZWQgcHJvdmlkZWQgdGhhdCB0aGUgZm9s
bG93aW5nIGNvbmRpdGlvbnMKKyMgYXJlIG1ldDoKKyMgMS4gUmVkaXN0cmlidXRpb25zIG9mIHNv
dXJjZSBjb2RlIG11c3QgcmV0YWluIHRoZSBhYm92ZSBjb3B5cmlnaHQKKyMgICAgbm90aWNlLCB0
aGlzIGxpc3Qgb2YgY29uZGl0aW9ucyBhbmQgdGhlIGZvbGxvd2luZyBkaXNjbGFpbWVyLgorIyAy
LiBSZWRpc3RyaWJ1dGlvbnMgaW4gYmluYXJ5IGZvcm0gbXVzdCByZXByb2R1Y2UgdGhlIGFib3Zl
IGNvcHlyaWdodAorIyAgICBub3RpY2UsIHRoaXMgbGlzdCBvZiBjb25kaXRpb25zIGFuZCB0aGUg
Zm9sbG93aW5nIGRpc2NsYWltZXIgaW4gdGhlCisjICAgIGRvY3VtZW50YXRpb24gYW5kL29yIG90
aGVyIG1hdGVyaWFscyBwcm92aWRlZCB3aXRoIHRoZSBkaXN0cmlidXRpb24uCisjCisjIFRISVMg
U09GVFdBUkUgSVMgUFJPVklERUQgQlkgVEhFIENPUFlSSUdIVCBIT0xERVJTIEFORCBDT05UUklC
VVRPUlMKKyMgIkFTIElTIiBBTkQgQU5ZIEVYUFJFU1MgT1IgSU1QTElFRCBXQVJSQU5USUVTLCBJ
TkNMVURJTkcsIEJVVCBOT1QKKyMgTElNSVRFRCBUTywgVEhFIElNUExJRUQgV0FSUkFOVElFUyBP
RiBNRVJDSEFOVEFCSUxJVFkgQU5EIEZJVE5FU1MgRk9SCisjIEEgUEFSVElDVUxBUiBQVVJQT1NF
IEFSRSBESVNDTEFJTUVELiBJTiBOTyBFVkVOVCBTSEFMTCBUSEUgQ09QWVJJR0hUCisjIE9XTkVS
IE9SIENPTlRSSUJVVE9SUyBCRSBMSUFCTEUgRk9SIEFOWSBESVJFQ1QsIElORElSRUNULCBJTkNJ
REVOVEFMLAorIyBTUEVDSUFMLCBFWEVNUExBUlksIE9SIENPTlNFUVVFTlRJQUwgREFNQUdFUyAo
SU5DTFVESU5HLCBCVVQgTk9UCisjIExJTUlURUQgVE8sIFBST0NVUkVNRU5UIE9GIFNVQlNUSVRV
VEUgR09PRFMgT1IgU0VSVklDRVM7IExPU1MgT0YgVVNFLAorIyBEQVRBLCBPUiBQUk9GSVRTOyBP
UiBCVVNJTkVTUyBJTlRFUlJVUFRJT04pIEhPV0VWRVIgQ0FVU0VEIEFORCBPTiBBTlkKKyMgVEhF
T1JZIE9GIExJQUJJTElUWSwgV0hFVEhFUiBJTiBDT05UUkFDVCwgU1RSSUNUIExJQUJJTElUWSwg
T1IgVE9SVAorIyAoSU5DTFVESU5HIE5FR0xJR0VOQ0UgT1IgT1RIRVJXSVNFKSBBUklTSU5HIElO
IEFOWSBXQVkgT1VUIE9GIFRIRSBVU0UKKyMgT0YgVEhJUyBTT0ZUV0FSRSwgRVZFTiBJRiBBRFZJ
U0VEIE9GIFRIRSBQT1NTSUJJTElUWSBPRiBTVUNIIERBTUFHRS4KKworIyEvYmluL2Jhc2gKKwor
ZWNobyBgVG9vbHMvU2NyaXB0cy9wcmVwYXJlLUNoYW5nZUxvZyAtLW5vLXdyaXRlIDI+JjEgfCBl
Z3JlcCAiXCguKlwpXDoiIHwgc2VkICJzLygvL2ciIHwgc2VkICJzLyk6Ly9nImAgfCBzZWQgInMv
IC9cbi9nIgo=
</data>
<flag name="review"
          id="129228"
          type_id="1"
          status="-"
          setter="darin"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>127335</attachid>
            <date>2012-02-16 02:33:21 -0800</date>
            <delta_ts>2012-02-24 10:41:50 -0800</delta_ts>
            <desc>Patch v2</desc>
            <filename>listmodifiedfunctions.diff</filename>
            <type>text/plain</type>
            <size>2358</size>
            <attacher name="Nandor Huszka">hnandor</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1Rvb2xzL0NoYW5nZUxvZyBiL1Rvb2xzL0NoYW5nZUxvZwppbmRleCBmZGEy
YmIxLi4xZjc1NjlmIDEwMDY0NAotLS0gYS9Ub29scy9DaGFuZ2VMb2cKKysrIGIvVG9vbHMvQ2hh
bmdlTG9nCkBAIC0xLDMgKzEsMTQgQEAKKzIwMTItMDItMTYgIE7DoW5kb3IgSHVzemthICA8aHVz
emthLm5hbmRvckBzdHVkLnUtc3plZ2VkLmh1PgorCisgICAgICAgIEFkZCBzY3JpcHQgdG8gZGV0
ZXJtaW5lIHdoaWNoIGZ1bmN0aW9ucyBhcmUgYWZmZWN0ZWQgYnkgYSBwYXRjaGFuYWx5c2lzdGFz
ay4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTc4NzAz
CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVXNlcyB0
aGUgcHJlcGFyZS1DaGFuZ2VMb2cncyAtLW5vLXdyaXRlIG9wdGlvbiB0byBkZXRlcm1pbmUgdGhl
IG1vZGlmaWVkIGZ1bmN0aW9ucy4KKworICAgICAgICAqIFNjcmlwdHMvbGlzdC1tb2RpZmllZC1m
dW5jdGlvbnM6IEFkZGVkLgorCiAyMDEyLTAyLTE2ICBQYXRyaWNrIEdhbnN0ZXJlciAgPHBhcm9n
YUB3ZWJraXQub3JnPgogCiAgICAgICAgIERpc2FibGUgaW5zcGVjdG9yIG9uIFdpbkNFLgpkaWZm
IC0tZ2l0IGEvVG9vbHMvU2NyaXB0cy9saXN0LW1vZGlmaWVkLWZ1bmN0aW9ucyBiL1Rvb2xzL1Nj
cmlwdHMvbGlzdC1tb2RpZmllZC1mdW5jdGlvbnMKaW5kZXggZTY5ZGUyOS4uMjc2NWI2NiAxMDA2
NDQKLS0tIGEvVG9vbHMvU2NyaXB0cy9saXN0LW1vZGlmaWVkLWZ1bmN0aW9ucworKysgYi9Ub29s
cy9TY3JpcHRzL2xpc3QtbW9kaWZpZWQtZnVuY3Rpb25zCkBAIC0wLDAgKzEsMjYgQEAKKyMgQ29w
eXJpZ2h0IChDKSAyMDEyIFVuaXZlcnNpdHkgb2YgU3plZ2VkLiBBbGwgcmlnaHRzIHJlc2VydmVk
LgorIworIyBSZWRpc3RyaWJ1dGlvbiBhbmQgdXNlIGluIHNvdXJjZSBhbmQgYmluYXJ5IGZvcm1z
LCB3aXRoIG9yIHdpdGhvdXQKKyMgbW9kaWZpY2F0aW9uLCBhcmUgcGVybWl0dGVkIHByb3ZpZGVk
IHRoYXQgdGhlIGZvbGxvd2luZyBjb25kaXRpb25zCisjIGFyZSBtZXQ6CisjIDEuIFJlZGlzdHJp
YnV0aW9ucyBvZiBzb3VyY2UgY29kZSBtdXN0IHJldGFpbiB0aGUgYWJvdmUgY29weXJpZ2h0Cisj
ICAgIG5vdGljZSwgdGhpcyBsaXN0IG9mIGNvbmRpdGlvbnMgYW5kIHRoZSBmb2xsb3dpbmcgZGlz
Y2xhaW1lci4KKyMgMi4gUmVkaXN0cmlidXRpb25zIGluIGJpbmFyeSBmb3JtIG11c3QgcmVwcm9k
dWNlIHRoZSBhYm92ZSBjb3B5cmlnaHQKKyMgICAgbm90aWNlLCB0aGlzIGxpc3Qgb2YgY29uZGl0
aW9ucyBhbmQgdGhlIGZvbGxvd2luZyBkaXNjbGFpbWVyIGluIHRoZQorIyAgICBkb2N1bWVudGF0
aW9uIGFuZC9vciBvdGhlciBtYXRlcmlhbHMgcHJvdmlkZWQgd2l0aCB0aGUgZGlzdHJpYnV0aW9u
LgorIworIyBUSElTIFNPRlRXQVJFIElTIFBST1ZJREVEIEJZIFRIRSBDT1BZUklHSFQgSE9MREVS
UyBBTkQgQ09OVFJJQlVUT1JTCisjICJBUyBJUyIgQU5EIEFOWSBFWFBSRVNTIE9SIElNUExJRUQg
V0FSUkFOVElFUywgSU5DTFVESU5HLCBCVVQgTk9UCisjIExJTUlURUQgVE8sIFRIRSBJTVBMSUVE
IFdBUlJBTlRJRVMgT0YgTUVSQ0hBTlRBQklMSVRZIEFORCBGSVRORVNTIEZPUgorIyBBIFBBUlRJ
Q1VMQVIgUFVSUE9TRSBBUkUgRElTQ0xBSU1FRC4gSU4gTk8gRVZFTlQgU0hBTEwgVEhFIENPUFlS
SUdIVAorIyBPV05FUiBPUiBDT05UUklCVVRPUlMgQkUgTElBQkxFIEZPUiBBTlkgRElSRUNULCBJ
TkRJUkVDVCwgSU5DSURFTlRBTCwKKyMgU1BFQ0lBTCwgRVhFTVBMQVJZLCBPUiBDT05TRVFVRU5U
SUFMIERBTUFHRVMgKElOQ0xVRElORywgQlVUIE5PVAorIyBMSU1JVEVEIFRPLCBQUk9DVVJFTUVO
VCBPRiBTVUJTVElUVVRFIEdPT0RTIE9SIFNFUlZJQ0VTOyBMT1NTIE9GIFVTRSwKKyMgREFUQSwg
T1IgUFJPRklUUzsgT1IgQlVTSU5FU1MgSU5URVJSVVBUSU9OKSBIT1dFVkVSIENBVVNFRCBBTkQg
T04gQU5ZCisjIFRIRU9SWSBPRiBMSUFCSUxJVFksIFdIRVRIRVIgSU4gQ09OVFJBQ1QsIFNUUklD
VCBMSUFCSUxJVFksIE9SIFRPUlQKKyMgKElOQ0xVRElORyBORUdMSUdFTkNFIE9SIE9USEVSV0lT
RSkgQVJJU0lORyBJTiBBTlkgV0FZIE9VVCBPRiBUSEUgVVNFCisjIE9GIFRISVMgU09GVFdBUkUs
IEVWRU4gSUYgQURWSVNFRCBPRiBUSEUgUE9TU0lCSUxJVFkgT0YgU1VDSCBEQU1BR0UuCisKKyMh
L2Jpbi9iYXNoCisKK2VjaG8gYFRvb2xzL1NjcmlwdHMvcHJlcGFyZS1DaGFuZ2VMb2cgLS1uby13
cml0ZSAyPiYxIHwgZWdyZXAgIlwoLipcKVw6IiB8IHNlZCAicy8oLy9nIiB8IHNlZCAicy8pOi8v
ZyJgIHwgc2VkICJzLyAvXG4vZyIK
</data>
<flag name="review"
          id="129433"
          type_id="1"
          status="-"
          setter="abarth"
    />
          </attachment>
      

    </bug>

</bugzilla>