<?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>19577</bug_id>
          
          <creation_ts>2008-06-16 06:34:24 -0700</creation_ts>
          <short_desc>JSCore does not compile in C++ environments where C99 headers are not present</short_desc>
          <delta_ts>2008-06-16 14:23:37 -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>Other</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>83451</commentid>
    <comment_count>0</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-06-16 06:34:24 -0700</bug_when>
    <thetext>Some JSCore public header files include stdbool.h. The stdbool.h header is a C99 feature, defining the &quot;_Bool&quot; type as well as the &quot;true&quot; and &quot;false&quot; constants. But it&apos;s completely unnecessary in C++ as the
language already defines the &quot;bool&quot; type and its two values.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>83452</commentid>
    <comment_count>1</comment_count>
      <attachid>21725</attachid>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-06-16 06:35:30 -0700</bug_when>
    <thetext>Created attachment 21725
Fix compilation in C++ environments where C99 headers are not present

Proposed patch from Thiago Macieira  &lt;tjmaciei@trolltech.com&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>83454</commentid>
    <comment_count>2</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-06-16 06:39:45 -0700</bug_when>
    <thetext>I think the patch is fine, but since this touches the header files of the public API I&apos;d be great to get a review from someone with experience on handling those :)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>83465</commentid>
    <comment_count>3</comment_count>
      <attachid>21725</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2008-06-16 08:53:35 -0700</bug_when>
    <thetext>Comment on attachment 21725
Fix compilation in C++ environments where C99 headers are not present

Seems OK. r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>83541</commentid>
    <comment_count>4</comment_count>
    <who name="Simon Hausmann">hausmann</who>
    <bug_when>2008-06-16 14:23:37 -0700</bug_when>
    <thetext>Landed in r34606</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>21725</attachid>
            <date>2008-06-16 06:35:30 -0700</date>
            <delta_ts>2008-06-16 08:53:35 -0700</delta_ts>
            <desc>Fix compilation in C++ environments where C99 headers are not present</desc>
            <filename>patch.txt</filename>
            <type>text/plain</type>
            <size>2583</size>
            <attacher name="Simon Hausmann">hausmann</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0FQSS9KU0Jhc2UuaCBiL0phdmFTY3JpcHRDb3Jl
L0FQSS9KU0Jhc2UuaAppbmRleCAyYzFlOWUzLi45MGRjNTYxIDEwMDY0NAotLS0gYS9KYXZhU2Ny
aXB0Q29yZS9BUEkvSlNCYXNlLmgKKysrIGIvSmF2YVNjcmlwdENvcmUvQVBJL0pTQmFzZS5oCkBA
IC0yNyw3ICsyNyw5IEBACiAjaWZuZGVmIEpTQmFzZV9oCiAjZGVmaW5lIEpTQmFzZV9oCiAKKyNp
Zm5kZWYgX19jcGx1c3BsdXMKICNpbmNsdWRlIDxzdGRib29sLmg+CisjZW5kaWYKIAogLyogSmF2
YVNjcmlwdCBlbmdpbmUgaW50ZXJmYWNlICovCiAKZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3Jl
L0FQSS9KU0NvbnRleHRSZWYuaCBiL0phdmFTY3JpcHRDb3JlL0FQSS9KU0NvbnRleHRSZWYuaApp
bmRleCBkZjBhMzk3Li5mODExZDIxIDEwMDY0NAotLS0gYS9KYXZhU2NyaXB0Q29yZS9BUEkvSlND
b250ZXh0UmVmLmgKKysrIGIvSmF2YVNjcmlwdENvcmUvQVBJL0pTQ29udGV4dFJlZi5oCkBAIC0z
MCw3ICszMCw5IEBACiAjaW5jbHVkZSA8SmF2YVNjcmlwdENvcmUvSlNPYmplY3RSZWYuaD4KICNp
bmNsdWRlIDxKYXZhU2NyaXB0Q29yZS9KU1ZhbHVlUmVmLmg+CiAKKyNpZm5kZWYgX19jcGx1c3Bs
dXMKICNpbmNsdWRlIDxzdGRib29sLmg+CisjZW5kaWYKIAogI2lmZGVmIF9fY3BsdXNwbHVzCiBl
eHRlcm4gIkMiIHsKZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0FQSS9KU09iamVjdFJlZi5o
IGIvSmF2YVNjcmlwdENvcmUvQVBJL0pTT2JqZWN0UmVmLmgKaW5kZXggYmY0MDQxZi4uZWU4ZmRj
YSAxMDA2NDQKLS0tIGEvSmF2YVNjcmlwdENvcmUvQVBJL0pTT2JqZWN0UmVmLmgKKysrIGIvSmF2
YVNjcmlwdENvcmUvQVBJL0pTT2JqZWN0UmVmLmgKQEAgLTMwLDcgKzMwLDkgQEAKICNpbmNsdWRl
IDxKYXZhU2NyaXB0Q29yZS9KU0Jhc2UuaD4KICNpbmNsdWRlIDxKYXZhU2NyaXB0Q29yZS9KU1Zh
bHVlUmVmLmg+CiAKKyNpZm5kZWYgX19jcGx1c3BsdXMKICNpbmNsdWRlIDxzdGRib29sLmg+Cisj
ZW5kaWYKICNpbmNsdWRlIDxzdGRkZWYuaD4gLy8gZm9yIHNpemVfdAogCiAjaWZkZWYgX19jcGx1
c3BsdXMKZGlmZiAtLWdpdCBhL0phdmFTY3JpcHRDb3JlL0FQSS9KU1N0cmluZ1JlZi5oIGIvSmF2
YVNjcmlwdENvcmUvQVBJL0pTU3RyaW5nUmVmLmgKaW5kZXggZmI1NjQ5Zi4uYzA5NTg4MiAxMDA2
NDQKLS0tIGEvSmF2YVNjcmlwdENvcmUvQVBJL0pTU3RyaW5nUmVmLmgKKysrIGIvSmF2YVNjcmlw
dENvcmUvQVBJL0pTU3RyaW5nUmVmLmgKQEAgLTI5LDcgKzI5LDkgQEAKIAogI2luY2x1ZGUgPEph
dmFTY3JpcHRDb3JlL0pTVmFsdWVSZWYuaD4KIAorI2lmbmRlZiBfX2NwbHVzcGx1cwogI2luY2x1
ZGUgPHN0ZGJvb2wuaD4KKyNlbmRpZgogI2luY2x1ZGUgPHN0ZGRlZi5oPiAvLyBmb3Igc2l6ZV90
CiAKICNpZmRlZiBfX2NwbHVzcGx1cwpkaWZmIC0tZ2l0IGEvSmF2YVNjcmlwdENvcmUvQVBJL0pT
VmFsdWVSZWYuaCBiL0phdmFTY3JpcHRDb3JlL0FQSS9KU1ZhbHVlUmVmLmgKaW5kZXggNmYxY2Uw
OC4uZGVhZGU3YyAxMDA2NDQKLS0tIGEvSmF2YVNjcmlwdENvcmUvQVBJL0pTVmFsdWVSZWYuaAor
KysgYi9KYXZhU2NyaXB0Q29yZS9BUEkvSlNWYWx1ZVJlZi5oCkBAIC0yOSw3ICsyOSw5IEBACiAK
ICNpbmNsdWRlIDxKYXZhU2NyaXB0Q29yZS9KU0Jhc2UuaD4KIAorI2lmbmRlZiBfX2NwbHVzcGx1
cwogI2luY2x1ZGUgPHN0ZGJvb2wuaD4KKyNlbmRpZgogCiAvKiEKIEBlbnVtIEpTVHlwZQpkaWZm
IC0tZ2l0IGEvSmF2YVNjcmlwdENvcmUvQ2hhbmdlTG9nIGIvSmF2YVNjcmlwdENvcmUvQ2hhbmdl
TG9nCmluZGV4IDk4YTYxZmEuLjZjMjQ4M2YgMTAwNjQ0Ci0tLSBhL0phdmFTY3JpcHRDb3JlL0No
YW5nZUxvZworKysgYi9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxOSBAQAor
MjAwOC0wNi0xNiAgVGhpYWdvIE1hY2llaXJhICA8dGptYWNpZWlAdHJvbGx0ZWNoLmNvbT4KKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKQorCisgICAgICAgIEZpeCBjb21waWxh
dGlvbiBpbiBDKysgZW52aXJvbm1lbnRzIHdoZXJlIEM5OSBoZWFkZXJzIGFyZSBub3QgcHJlc2Vu
dAorCisgICAgICAgIFRoZSBzdGRib29sLmggaGVhZGVyIGlzIGEgQzk5IGZlYXR1cmUsIGRlZmlu
aW5nIHRoZSAiX0Jvb2wiIHR5cGUgYXMgd2VsbCBhcyB0aGUKKyAgICAgICAgInRydWUiIGFuZCAi
ZmFsc2UiIGNvbnN0YW50cy4gQnV0IGl0J3MgY29tcGxldGVseSB1bm5lY2Vzc2FyeSBpbiBDKysg
YXMgdGhlCisgICAgICAgIGxhbmd1YWdlIGFscmVhZHkgZGVmaW5lcyB0aGUgImJvb2wiIHR5cGUg
YW5kIGl0cyB0d28gdmFsdWVzLgorCisgICAgICAgICogQVBJL0pTQmFzZS5oOgorICAgICAgICAq
IEFQSS9KU0NvbnRleHRSZWYuaDoKKyAgICAgICAgKiBBUEkvSlNPYmplY3RSZWYuaDoKKyAgICAg
ICAgKiBBUEkvSlNTdHJpbmdSZWYuaDoKKyAgICAgICAgKiBBUEkvSlNWYWx1ZVJlZi5oOgorCiAy
MDA4LTA2LTE2ICBBZHJpYWFuIGRlIEdyb290ICA8Z3Jvb3RAa2RlLm9yZz4KIAogICAgICAgICBS
ZXZpZXdlZCBieSBTaW1vbi4K
</data>
<flag name="review"
          id="9545"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>