<?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>145922</bug_id>
          
          <creation_ts>2015-06-12 04:26:11 -0700</creation_ts>
          <short_desc>[GTK] Fix deprecated-register warning in the generated WebKitMarshal.cpp</short_desc>
          <delta_ts>2015-06-15 06:10:22 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          <see_also>https://bugzilla.gnome.org/show_bug.cgi?id=750918</see_also>
          <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>145121</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Csaba Osztrogonác">ossy</reporter>
          <assigned_to name="Csaba Osztrogonác">ossy</assigned_to>
          <cc>cgarcia</cc>
    
    <cc>commit-queue</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>mrobinson</cc>
    
    <cc>ossy</cc>
    
    <cc>zan</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1101549</commentid>
    <comment_count>0</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2015-06-12 04:26:11 -0700</bug_when>
    <thetext>DerivedSources/webkit2gtk/webkit2/WebKitMarshal.cpp:65:3: warning: &apos;register&apos; storage class specifier is deprecated [-Wdeprecated-register]
  register GMarshalFunc_BOOLEAN__BOXED callback;
  ^~~~~~~~~
DerivedSources/webkit2gtk/webkit2/WebKitMarshal.cpp:66:3: warning: &apos;register&apos; storage class specifier is deprecated [-Wdeprecated-register]
  register GCClosure *cc = (GCClosure*) closure;
  ^~~~~~~~~

...

The problem is that WebKitMarshal.cpp is generated by glib-genmarshal. To fix 
it properly, we should fix it in upstream or patch the glib jhbuild module.
Or we should simply disable this warning for this file only.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1101857</commentid>
    <comment_count>1</comment_count>
    <who name="Michael Catanzaro">mcatanzaro</who>
    <bug_when>2015-06-14 09:59:52 -0700</bug_when>
    <thetext>(In reply to comment #0)
&gt; To fix it properly, we should fix it in upstream

Done. I usually build inside GNOME jhbuild, so the problem is gone for me :)

&gt; or patch the glib jhbuild module. Or we should simply disable this warning for this file only.

^ But to fix the issue with WebKit jhbuild, we still need to do one of the above.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1101907</commentid>
    <comment_count>2</comment_count>
      <attachid>254872</attachid>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2015-06-15 04:22:02 -0700</bug_when>
    <thetext>Created attachment 254872
Patch

Thanks for the quick fix. I prefer adding your upstream patch to the WebKit&apos;s jhbuild</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1101914</commentid>
    <comment_count>3</comment_count>
      <attachid>254872</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2015-06-15 06:10:18 -0700</bug_when>
    <thetext>Comment on attachment 254872
Patch

Clearing flags on attachment: 254872

Committed r185550: &lt;http://trac.webkit.org/changeset/185550&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1101915</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2015-06-15 06:10:22 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>254872</attachid>
            <date>2015-06-15 04:22:02 -0700</date>
            <delta_ts>2015-06-15 06:10:18 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-145922-20150615042127.patch</filename>
            <type>text/plain</type>
            <size>3122</size>
            <attacher name="Csaba Osztrogonác">ossy</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTg1NTQ3CmRpZmYgLS1naXQgYS9Ub29scy9DaGFuZ2VMb2cg
Yi9Ub29scy9DaGFuZ2VMb2cKaW5kZXggMjFlODkzY2NhNGJlZmRjYzgwYjY1ZWE5ZDQ4YTA3ZGY4
NjdkNTgzOS4uOWYyM2IyMjUzODFmZDI3MGM5ZmJlN2YwYTY0MGNlODE0YTk0NjZjNSAxMDA2NDQK
LS0tIGEvVG9vbHMvQ2hhbmdlTG9nCisrKyBiL1Rvb2xzL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEz
IEBACisyMDE1LTA2LTE1ICBDc2FiYSBPc3p0cm9nb27DoWMgIDxvc3N5QHdlYmtpdC5vcmc+CisK
KyAgICAgICAgW0dUS10gRml4IGRlcHJlY2F0ZWQtcmVnaXN0ZXIgd2FybmluZyBpbiB0aGUgZ2Vu
ZXJhdGVkIFdlYktpdE1hcnNoYWwuY3BwCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3Jn
L3Nob3dfYnVnLmNnaT9pZD0xNDU5MjIKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9P
UFMhKS4KKworICAgICAgICAqIGd0ay9qaGJ1aWxkLm1vZHVsZXM6CisgICAgICAgICogZ3RrL3Bh
dGNoZXMvZ2xpYi13YXJuaW5nLWZpeC5wYXRjaDogQWRkZWQuCisKIDIwMTUtMDYtMTIgIEFsZXhl
eSBQcm9za3VyeWFrb3YgIDxhcEBhcHBsZS5jb20+CiAKICAgICAgICAgVXBkYXRlIHNwcmluZ0xv
YWRpbmdIaWdobGlnaHQgdHlwZQpkaWZmIC0tZ2l0IGEvVG9vbHMvZ3RrL2poYnVpbGQubW9kdWxl
cyBiL1Rvb2xzL2d0ay9qaGJ1aWxkLm1vZHVsZXMKaW5kZXggMmU2YmU5ODJkMjcxOTA2YTAzYWNm
NjRiY2MyY2I2NmU1NDE1OWI4Ny4uNjEyMTI4ZTBhYjI4ZjdmYmM3YjgxNGY2MmM5YTAwNDc2ZTcw
NDFjMCAxMDA2NDQKLS0tIGEvVG9vbHMvZ3RrL2poYnVpbGQubW9kdWxlcworKysgYi9Ub29scy9n
dGsvamhidWlsZC5tb2R1bGVzCkBAIC0xNzIsNyArMTcyLDkgQEAKICAgICA8YnJhbmNoIG1vZHVs
ZT0iL3B1Yi9HTk9NRS9zb3VyY2VzL2dsaWIvMi40MS9nbGliLTIuNDEuMi50YXIueHoiIHZlcnNp
b249IjIuNDEuMiIKICAgICAgICAgICAgIHJlcG89ImZ0cC5nbm9tZS5vcmciCiAgICAgICAgICAg
ICBoYXNoPSJzaGEyNTY6ZGExZjcyNTg2NTVmMGUxOTZiOTMwNGNiOTUxNTc4NDYzNGYzNzk3NDM3
OTIzZTIzNmJiMzQ2NjM0ODgxMWM5NiIKLSAgICAgICAgICAgIG1kNXN1bT0iOTYzNmY2MGU5OWI5
OGZkMGZkYjUyMzlmOTA1YWMwMDgiLz4KKyAgICAgICAgICAgIG1kNXN1bT0iOTYzNmY2MGU5OWI5
OGZkMGZkYjUyMzlmOTA1YWMwMDgiPgorICAgICAgPHBhdGNoIGZpbGU9ImdsaWItd2FybmluZy1m
aXgucGF0Y2giIHN0cmlwPSIxIi8+CisgICAgPC9icmFuY2g+CiAgIDwvYXV0b3Rvb2xzPgogCiAg
IDxhdXRvdG9vbHMgaWQ9ImdsaWItbmV0d29ya2luZyI+CmRpZmYgLS1naXQgYS9Ub29scy9ndGsv
cGF0Y2hlcy9nbGliLXdhcm5pbmctZml4LnBhdGNoIGIvVG9vbHMvZ3RrL3BhdGNoZXMvZ2xpYi13
YXJuaW5nLWZpeC5wYXRjaApuZXcgZmlsZSBtb2RlIDEwMDY0NAppbmRleCAwMDAwMDAwMDAwMDAw
MDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwMDAwLi44ZmFmNGQxMWYyY2M1NWE0NDJiMzhjZGQyY2M0
NWZkMzI4OTZlMjNjCi0tLSAvZGV2L251bGwKKysrIGIvVG9vbHMvZ3RrL3BhdGNoZXMvZ2xpYi13
YXJuaW5nLWZpeC5wYXRjaApAQCAtMCwwICsxLDM0IEBACitGcm9tIDlmOTBlZTVlZWNjZDQ3ZjM5
YzdhMDNkY2Q3ODZiMTI1YTE5YzE5NWQgTW9uIFNlcCAxNyAwMDowMDowMCAyMDAxCitGcm9tOiBN
aWNoYWVsIENhdGFuemFybyA8bWNhdGFuemFyb0Bnbm9tZS5vcmc+CitEYXRlOiBTYXQsIDEzIEp1
biAyMDE1IDIyOjUyOjMzIC0wNTAwCitTdWJqZWN0OiBbUEFUQ0hdIGdlbm1hcnNoYWw6IHNpbGVu
Y2UgcmVnaXN0ZXIgc3RvcmFnZSBjbGFzcyB3YXJuaW5ncworCitVc2luZyB0aGUgcmVnaXN0ZXIg
a2V5d29yZCB0cmlnZ2VycyB3YXJuaW5ncyBvbiBub3Rld29ydGh5IGNvbXBpbGVycworKGNsYW5n
KSwgc2luY2UgaXQncyBkZXByZWNhdGVkIGluIEMrKyBhbmQgYXQgZGFuZ2VyIG9mIGJlaW5nIHJl
bW92ZWQKK2Zyb20gdGhlIGxhbmd1YWdlLiBUaGVyZSBpcyBubyByZWFzb24gdG8gdXNlIGl0IHNp
bmNlIGl0IGlzbid0IDE5ODAKK2FueW1vcmUuCisKK2h0dHBzOi8vYnVnemlsbGEuZ25vbWUub3Jn
L3Nob3dfYnVnLmNnaT9pZD03NTA5MTgKKy0tLQorIGdvYmplY3QvZ2xpYi1nZW5tYXJzaGFsLmMg
fCA2ICsrKy0tLQorIDEgZmlsZSBjaGFuZ2VkLCAzIGluc2VydGlvbnMoKyksIDMgZGVsZXRpb25z
KC0pCisKK2RpZmYgLS1naXQgYS9nb2JqZWN0L2dsaWItZ2VubWFyc2hhbC5jIGIvZ29iamVjdC9n
bGliLWdlbm1hcnNoYWwuYworaW5kZXggYmU0MTUxYS4uY2E3OGE2ZiAxMDA2NDQKKy0tLSBhL2dv
YmplY3QvZ2xpYi1nZW5tYXJzaGFsLmMKKysrKyBiL2dvYmplY3QvZ2xpYi1nZW5tYXJzaGFsLmMK
K0BAIC00MTIsOSArNDEyLDkgQEAgZ2VuZXJhdGVfbWFyc2hhbCAoY29uc3QgZ2NoYXIgKnNpZ25h
bWUsCisgICAgICAgZ19mcHJpbnRmIChmb3V0LCAiJXMlcyBkYXRhMik7XG4iLCBpbmRlbnQgKGlu
ZCksIHBhZCAoImdwb2ludGVyIikpOworIAorICAgICAgIC8qIGNmaWxlIG1hcnNoYWwgdmFyaWFi
bGVzICovCistICAgICAgZ19mcHJpbnRmIChmb3V0LCAiICByZWdpc3RlciBHTWFyc2hhbEZ1bmNf
JXMgY2FsbGJhY2s7XG4iLCBzaWduYW1lKTsKKy0gICAgICBnX2ZwcmludGYgKGZvdXQsICIgIHJl
Z2lzdGVyIEdDQ2xvc3VyZSAqY2MgPSAoR0NDbG9zdXJlKikgY2xvc3VyZTtcbiIpOworLSAgICAg
IGdfZnByaW50ZiAoZm91dCwgIiAgcmVnaXN0ZXIgZ3BvaW50ZXIgZGF0YTEsIGRhdGEyO1xuIik7
CisrICAgICAgZ19mcHJpbnRmIChmb3V0LCAiICBHTWFyc2hhbEZ1bmNfJXMgY2FsbGJhY2s7XG4i
LCBzaWduYW1lKTsKKysgICAgICBnX2ZwcmludGYgKGZvdXQsICIgIEdDQ2xvc3VyZSAqY2MgPSAo
R0NDbG9zdXJlKikgY2xvc3VyZTtcbiIpOworKyAgICAgIGdfZnByaW50ZiAoZm91dCwgIiAgZ3Bv
aW50ZXIgZGF0YTEsIGRhdGEyO1xuIik7CisgICAgICAgaWYgKHNpZy0+cmFyZy0+c2V0dGVyKQor
IAlnX2ZwcmludGYgKGZvdXQsICIgICVzIHZfcmV0dXJuO1xuIiwgc2lnLT5yYXJnLT5jdHlwZSk7
CisgCistLSAKKzIuNC4yClwgTm8gbmV3bGluZSBhdCBlbmQgb2YgZmlsZQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>