<?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>90681</bug_id>
          
          <creation_ts>2012-07-06 06:28:48 -0700</creation_ts>
          <short_desc>[EFL] WebKit-EFL headers do not build with gcc &lt; 4.6</short_desc>
          <delta_ts>2012-07-06 07:16:27 -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>WebKit EFL</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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="Chris Dumez">cdumez</reporter>
          <assigned_to name="Chris Dumez">cdumez</assigned_to>
          <cc>gyuyoung.kim</cc>
    
    <cc>lucas.de.marchi</cc>
    
    <cc>rakuco</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>664066</commentid>
    <comment_count>0</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2012-07-06 06:28:48 -0700</bug_when>
    <thetext>Because of forward declarations of our struct types in the public headers, we end up with redefinitions of typedefs. While this is allowed in C++, it is not valid C.
We need to use #includes in the public headers instead of forwards declarations.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>664068</commentid>
    <comment_count>1</comment_count>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2012-07-06 06:31:58 -0700</bug_when>
    <thetext>gcc 4.6 started adding some features of upcoming C1X revision of the ISO C standard, which is why our MiniBrowser compiles:
http://gcc.gnu.org/gcc-4.6/changes.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>664072</commentid>
    <comment_count>2</comment_count>
      <attachid>151076</attachid>
    <who name="Chris Dumez">cdumez</who>
    <bug_when>2012-07-06 06:34:31 -0700</bug_when>
    <thetext>Created attachment 151076
Patch

Could someone please cq+ ?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>664097</commentid>
    <comment_count>3</comment_count>
      <attachid>151076</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-07-06 07:16:23 -0700</bug_when>
    <thetext>Comment on attachment 151076
Patch

Clearing flags on attachment: 151076

Committed r121969: &lt;http://trac.webkit.org/changeset/121969&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>664098</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-07-06 07:16:27 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>151076</attachid>
            <date>2012-07-06 06:34:31 -0700</date>
            <delta_ts>2012-07-06 07:16:23 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>90681_build_fix.patch</filename>
            <type>text/plain</type>
            <size>1520</size>
            <attacher name="Chris Dumez">cdumez</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQyL0No
YW5nZUxvZwppbmRleCA1OTc5NTg2Li4yOTg4ZDMyIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0
Mi9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTYg
QEAKKzIwMTItMDctMDYgIENocmlzdG9waGUgRHVtZXogIDxjaHJpc3RvcGhlLmR1bWV6QGludGVs
LmNvbT4KKworICAgICAgICBbRUZMXSBXZWJLaXQtRUZMIGhlYWRlcnMgZG8gbm90IGJ1aWxkIHdp
dGggZ2NjIDwgNC42CisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD05MDY4MQorCisgICAgICAgIFVucmV2aWV3ZWQgRUZMIGJ1aWxkIGZpeC4KKworICAgICAg
ICBSZW1vdmUgc2V2ZXJhbCBmb3J3YXJkIGRlY2xhcmF0aW9ucyBmcm9tIGV3a192aWV3CisgICAg
ICAgIGhlYWRlciB0byBhdm9pZCB0eXBlZGVmIHJlZGVmaW5pdGlvbnMgd2hpY2ggYXJlCisgICAg
ICAgIGlsbGVnYWwgaW4gQy4KKworICAgICAgICAqIFVJUHJvY2Vzcy9BUEkvZWZsL2V3a192aWV3
Lmg6CisKIDIwMTItMDctMDYgIE9zd2FsZCBCdWRkZW5oYWdlbiAgPG9zd2FsZC5idWRkZW5oYWdl
bkBub2tpYS5jb20+CiAKICAgICAgICAgW1F0XSBBZGp1c3QgdG8gY2hhbmdlZCBnZW5lcmF0aW9u
IG9mIG1hc3RlciBpbmNsdWRlIGZpbGUKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJv
Y2Vzcy9BUEkvZWZsL2V3a192aWV3LmggYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL2Vm
bC9ld2tfdmlldy5oCmluZGV4IGExYTNhN2QuLmY2MDkyYTYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9X
ZWJLaXQyL1VJUHJvY2Vzcy9BUEkvZWZsL2V3a192aWV3LmgKKysrIGIvU291cmNlL1dlYktpdDIv
VUlQcm9jZXNzL0FQSS9lZmwvZXdrX3ZpZXcuaApAQCAtNDIsNiArNDIsOSBAQAogI2RlZmluZSBl
d2tfdmlld19oCiAKICNpbmNsdWRlICJld2tfY29udGV4dC5oIgorI2luY2x1ZGUgImV3a19pbnRl
bnQuaCIKKyNpbmNsdWRlICJld2tfdXJsX3JlcXVlc3QuaCIKKyNpbmNsdWRlICJld2tfd2ViX3Jl
c291cmNlLmgiCiAjaW5jbHVkZSA8RXZhcy5oPgogCiAjaWZkZWYgX19jcGx1c3BsdXMKQEAgLTUw
LDEwICs1Myw2IEBAIGV4dGVybiAiQyIgewogCiB0eXBlZGVmIHN0cnVjdCBfRXdrX1ZpZXdfU21h
cnRfRGF0YSBFd2tfVmlld19TbWFydF9EYXRhOwogdHlwZWRlZiBzdHJ1Y3QgX0V3a19WaWV3X1Nt
YXJ0X0NsYXNzIEV3a19WaWV3X1NtYXJ0X0NsYXNzOwotdHlwZWRlZiBzdHJ1Y3QgX0V3a19JbnRl
bnQgRXdrX0ludGVudDsKLXR5cGVkZWYgc3RydWN0IF9Fd2tfVXJsX1JlcXVlc3QgRXdrX1VybF9S
ZXF1ZXN0OwotdHlwZWRlZiBzdHJ1Y3QgX0V3a19XZWJfUmVzb3VyY2UgRXdrX1dlYl9SZXNvdXJj
ZTsKLQogCiAvLy8gRXdrIHZpZXcncyBjbGFzcywgdG8gYmUgb3ZlcnJpZGRlbiBieSBzdWItY2xh
c3Nlcy4KIHN0cnVjdCBfRXdrX1ZpZXdfU21hcnRfQ2xhc3Mgewo=
</data>

          </attachment>
      

    </bug>

</bugzilla>