<?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>27055</bug_id>
          
          <creation_ts>2009-07-07 19:03:20 -0700</creation_ts>
          <short_desc>Qualify include path with wtf</short_desc>
          <delta_ts>2009-07-15 08:14:25 -0700</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>S60 Hardware</rep_platform>
          <op_sys>S60 3rd edition</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>27065</blocked>
          <everconfirmed>0</everconfirmed>
          <reporter name="Norbert Leser">norbert.leser</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>bfulgham</cc>
    
    <cc>hausmann</cc>
    
    <cc>laszlo.gombos</cc>
    
    <cc>yongjun.zhang</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>130336</commentid>
    <comment_count>0</comment_count>
      <attachid>32415</attachid>
    <who name="Norbert Leser">norbert.leser</who>
    <bug_when>2009-07-07 19:03:20 -0700</bug_when>
    <thetext>Created attachment 32415
Code patch for Interpreter.h

On Symbian platforms we encounter include ambiguities of HashMap.h file, which is already present in default location.

Since that header filename is very generic, it appears to be appropriate and saver to add a pathname qualifier &quot;wtf/HashMap.h&quot; to the include statement.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>130360</commentid>
    <comment_count>1</comment_count>
      <attachid>32415</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2009-07-07 21:59:39 -0700</bug_when>
    <thetext>Comment on attachment 32415
Code patch for Interpreter.h

It&apos;s OK to change the include here to the normal style, but the normal style is to include &lt;wtf/HashMap.h&gt; and include in a separate paragraph.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131263</commentid>
    <comment_count>2</comment_count>
      <attachid>32415</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-07-13 00:45:54 -0700</bug_when>
    <thetext>Comment on attachment 32415
Code patch for Interpreter.h

r- per Darin&apos;s comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131783</commentid>
    <comment_count>3</comment_count>
      <attachid>32756</attachid>
    <who name="Laszlo Gombos">laszlo.gombos</who>
    <bug_when>2009-07-14 18:25:31 -0700</bug_when>
    <thetext>Created attachment 32756
Second try.

Updated the patch based on Darin&apos;s feedback (filling in for Norbert.)
Note that since this patch reverses a subset of r44418 (presumably a build fix for WIN_CAIRO), the proposed fix might break WIN_CAIRO port (I CCd Brent on the bug).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131801</commentid>
    <comment_count>4</comment_count>
    <who name="Brent Fulgham">bfulgham</who>
    <bug_when>2009-07-14 20:20:08 -0700</bug_when>
    <thetext>This change should be fine for WIN_CAIRO.  I added the HashMap.h because it was needed to compile.  Using the fully-qualified wtf/HashMap.h path should not be a problem.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>131890</commentid>
    <comment_count>5</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2009-07-15 08:14:25 -0700</bug_when>
    <thetext>Landed in r45909</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>32415</attachid>
            <date>2009-07-07 19:03:20 -0700</date>
            <delta_ts>2009-07-13 00:45:54 -0700</delta_ts>
            <desc>Code patch for Interpreter.h</desc>
            <filename>patch_Interpreter_h.txt</filename>
            <type>text/plain</type>
            <size>952</size>
            <attacher name="Norbert Leser">norbert.leser</attacher>
            
              <data encoding="base64">SW5kZXg6IENoYW5nZUxvZw0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PQ0KLS0tIENoYW5nZUxvZwkocmV2aXNpb24gNDU2
MDgpCisrKyBDaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxMiBAQAorMjAwOS0w
Ny0wNyAgTm9yYmVydCBMZXNlciAgPG5vcmJlcnQubGVzZXJAbm9raWEuY29tPgorCisgICAgICAg
IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFF1YWxpZnkgaW5jbHVkZSBw
YXRoIHdpdGggd3RmLiAgICAgICAgCisgICAgICAgIEF2b2lkcyBpbmNsdWRlIGFtYmlndWl0aWVz
IG9mIEhhc2hNYXAuaCBmaWxlLgorCisgICAgICAgICogSmF2YVNjcmlwdENvcmUvaW50ZXJwcmV0
ZXIvSW50ZXJwcmV0ZXIuaDoKKwogMjAwOS0wNy0wMyAgSmFuIE1pY2hhZWwgQWxvbnpvICA8am1h
bG9uem9Ad2Via2l0Lm9yZz4KIAogICAgICAgICBSZXZpZXdlZCBieSBYYW4gTG9wZXogYW5kIEd1
c3Rhdm8gTm9yb25oYS4KSW5kZXg6IEphdmFTY3JpcHRDb3JlL2ludGVycHJldGVyL0ludGVycHJl
dGVyLmgNCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT0NCi0tLSBKYXZhU2NyaXB0Q29yZS9pbnRlcnByZXRlci9JbnRlcnBy
ZXRlci5oCShyZXZpc2lvbiA0NTYwOCkKKysrIEphdmFTY3JpcHRDb3JlL2ludGVycHJldGVyL0lu
dGVycHJldGVyLmgJKHdvcmtpbmcgY29weSkKQEAgLTMxLDcgKzMxLDcgQEAKIAogI2luY2x1ZGUg
IkFyZ0xpc3QuaCIKICNpbmNsdWRlICJGYXN0QWxsb2NCYXNlLmgiCi0jaW5jbHVkZSAiSGFzaE1h
cC5oIgorI2luY2x1ZGUgInd0Zi9IYXNoTWFwLmgiCiAjaW5jbHVkZSAiSlNDZWxsLmgiCiAjaW5j
bHVkZSAiSlNWYWx1ZS5oIgogI2luY2x1ZGUgIkpTT2JqZWN0LmgiCg==
</data>
<flag name="review"
          id="16863"
          type_id="1"
          status="-"
          setter="eric"
    />
          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>32756</attachid>
            <date>2009-07-14 18:25:31 -0700</date>
            <delta_ts>2009-07-14 21:58:00 -0700</delta_ts>
            <desc>Second try.</desc>
            <filename>patch_27055.txt</filename>
            <type>text/plain</type>
            <size>1046</size>
            <attacher name="Laszlo Gombos">laszlo.gombos</attacher>
            
              <data encoding="base64">SW5kZXg6IEphdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDQ1ODg3KQorKysgSmF2YVNjcmlwdENvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTIgQEAKKzIwMDktMDctMTQgIE5vcmJlcnQg
TGVzZXIgIDxub3JiZXJ0Lmxlc2VyQG5va2lhLmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBO
T0JPRFkgKE9PUFMhKS4KKworICAgICAgICBRdWFsaWZ5IGluY2x1ZGUgcGF0aCB3aXRoIHd0Zgor
ICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MjcwNTUKKwor
ICAgICAgICAqIGludGVycHJldGVyL0ludGVycHJldGVyLmg6CisKIDIwMDktMDctMTQgIFpvbHRh
biBIb3J2YXRoICA8aHpvbHRhbkBpbmYudS1zemVnZWQuaHU+CiAKICAgICAgICAgUmV2aWV3ZWQg
YnkgRGFyaW4gQWRsZXIuCkluZGV4OiBKYXZhU2NyaXB0Q29yZS9pbnRlcnByZXRlci9JbnRlcnBy
ZXRlci5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT0KLS0tIEphdmFTY3JpcHRDb3JlL2ludGVycHJldGVyL0ludGVycHJl
dGVyLmgJKHJldmlzaW9uIDQ1ODU2KQorKysgSmF2YVNjcmlwdENvcmUvaW50ZXJwcmV0ZXIvSW50
ZXJwcmV0ZXIuaAkod29ya2luZyBjb3B5KQpAQCAtMzEsMTMgKzMxLDE0IEBACiAKICNpbmNsdWRl
ICJBcmdMaXN0LmgiCiAjaW5jbHVkZSAiRmFzdEFsbG9jQmFzZS5oIgotI2luY2x1ZGUgIkhhc2hN
YXAuaCIKICNpbmNsdWRlICJKU0NlbGwuaCIKICNpbmNsdWRlICJKU1ZhbHVlLmgiCiAjaW5jbHVk
ZSAiSlNPYmplY3QuaCIKICNpbmNsdWRlICJPcGNvZGUuaCIKICNpbmNsdWRlICJSZWdpc3RlckZp
bGUuaCIKIAorI2luY2x1ZGUgPHd0Zi9IYXNoTWFwLmg+CisKIG5hbWVzcGFjZSBKU0MgewogCiAg
ICAgY2xhc3MgQ29kZUJsb2NrOwo=
</data>
<flag name="review"
          id="17129"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>