<?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>182642</bug_id>
          
          <creation_ts>2018-02-09 08:53:49 -0800</creation_ts>
          <short_desc>Improve of string.h include after r228317.</short_desc>
          <delta_ts>2018-02-09 10:04:26 -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>bmalloc</component>
          <version>WebKit 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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Carlos Alberto Lopez Perez">clopez</reporter>
          <assigned_to name="Carlos Alberto Lopez Perez">clopez</assigned_to>
          <cc>clopez</cc>
    
    <cc>fpizlo</cc>
    
    <cc>ggaren</cc>
    
    <cc>mark.lam</cc>
    
    <cc>mcatanzaro</cc>
    
    <cc>msaboff</cc>
    
    <cc>saam</cc>
    
    <cc>webkit-bug-importer</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1397952</commentid>
    <comment_count>0</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2018-02-09 08:53:49 -0800</bug_when>
    <thetext>From https://bugs.webkit.org/show_bug.cgi?id=182563#c22
&gt; (In reply to Carlos Alberto Lopez Perez from comment #20)
&gt; &gt; Committed r228317: &lt;https://trac.webkit.org/changeset/228317&gt;
&gt; 
&gt; We should avoid architecture-specific #includes whenever possible. Adding
&gt; one #include &lt;string.h&gt; is not going to slow down the build very much at
&gt; all; it&apos;s not worth the cost of the include guards, IMO.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1397954</commentid>
    <comment_count>1</comment_count>
      <attachid>333493</attachid>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2018-02-09 09:04:29 -0800</bug_when>
    <thetext>Created attachment 333493
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1397956</commentid>
    <comment_count>2</comment_count>
      <attachid>333494</attachid>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2018-02-09 09:06:16 -0800</bug_when>
    <thetext>Created attachment 333494
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1397988</commentid>
    <comment_count>3</comment_count>
      <attachid>333494</attachid>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2018-02-09 10:03:07 -0800</bug_when>
    <thetext>Comment on attachment 333494
Patch

Clearing flags on attachment: 333494

Committed r228324: &lt;https://trac.webkit.org/changeset/228324&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1397989</commentid>
    <comment_count>4</comment_count>
    <who name="Carlos Alberto Lopez Perez">clopez</who>
    <bug_when>2018-02-09 10:03:11 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1397993</commentid>
    <comment_count>5</comment_count>
    <who name="Radar WebKit Bug Importer">webkit-bug-importer</who>
    <bug_when>2018-02-09 10:04:26 -0800</bug_when>
    <thetext>&lt;rdar://problem/37394750&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>333493</attachid>
            <date>2018-02-09 09:04:29 -0800</date>
            <delta_ts>2018-02-09 09:06:09 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-182642-20180209180428.patch</filename>
            <type>text/plain</type>
            <size>1208</size>
            <attacher name="Carlos Alberto Lopez Perez">clopez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjI4MzE4CmRpZmYgLS1naXQgYS9Tb3VyY2UvYm1hbGxvYy9D
aGFuZ2VMb2cgYi9Tb3VyY2UvYm1hbGxvYy9DaGFuZ2VMb2cKaW5kZXggYzVhN2JjOTFjZjExNmQ0
YWE5Y2M1YTAyYzI5ODA5NWEzMTJhNjNmNC4uZTI1YzExYmVkZmYwM2RlN2M5NzJiOGQxMzUyZDI0
YjA1ZDY0NzliYyAxMDA2NDQKLS0tIGEvU291cmNlL2JtYWxsb2MvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9ibWFsbG9jL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE0IEBACiAyMDE4LTAyLTA5ICBDYXJs
b3MgQWxiZXJ0byBMb3BleiBQZXJleiAgPGNsb3BlekBpZ2FsaWEuY29tPgogCisgICAgICAgIElt
cHJvdmUgb2Ygc3RyaW5nLmggaW5jbHVkZSBhZnRlciByMjI4MzE3LgorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTgyNjQyCisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBibWFsbG9jL0FsZ29yaXRobS5oOiBB
dm9pZCBhbiBhcmNoaXRlY3R1cmUtc3BlY2lmaWMgI2luY2x1ZGUuCisKKzIwMTgtMDItMDkgIENh
cmxvcyBBbGJlcnRvIExvcGV6IFBlcmV6ICA8Y2xvcGV6QGlnYWxpYS5jb20+CisKICAgICAgICAg
Rml4IGJ1aWxkIGZvciAhQkNQVShYODZfNjQpIGFmdGVyIHIyMjgzMDYKICAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE4MjU2MwogCmRpZmYgLS1naXQgYS9T
b3VyY2UvYm1hbGxvYy9ibWFsbG9jL0FsZ29yaXRobS5oIGIvU291cmNlL2JtYWxsb2MvYm1hbGxv
Yy9BbGdvcml0aG0uaAppbmRleCA4NmE1ZjkyYmFhY2JjZmQ4ZDAyNzkwMGVlNDZmNjg2OTA1YzEz
MmFkLi43NzI0MTMyYzU4MzZlZDA5ZjJmZWU0OThkZTBiZGE2NmU5ZGRmMWQ1IDEwMDY0NAotLS0g
YS9Tb3VyY2UvYm1hbGxvYy9ibWFsbG9jL0FsZ29yaXRobS5oCisrKyBiL1NvdXJjZS9ibWFsbG9j
L2JtYWxsb2MvQWxnb3JpdGhtLmgKQEAgLTMxLDEyICszMSwxMCBAQAogI2luY2x1ZGUgPGNzdGRp
bnQ+CiAjaW5jbHVkZSA8Y3N0ZGRlZj4KICNpbmNsdWRlIDxsaW1pdHM+CisjaW5jbHVkZSA8c3Ry
aW5nLmg+CiAjaW5jbHVkZSA8dHlwZV90cmFpdHM+CiAjaW5jbHVkZSA8Y2hyb25vPgogCi0jaWYg
IUJDUFUoWDg2XzY0KQotI2luY2x1ZGUgPHN0cmluZy5oPgotI2VuZGlmCiAKIG5hbWVzcGFjZSBi
bWFsbG9jIHsKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>333494</attachid>
            <date>2018-02-09 09:06:16 -0800</date>
            <delta_ts>2018-02-09 10:03:07 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-182642-20180209180615.patch</filename>
            <type>text/plain</type>
            <size>1285</size>
            <attacher name="Carlos Alberto Lopez Perez">clopez</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMjI4MzE4CmRpZmYgLS1naXQgYS9Tb3VyY2UvYm1hbGxvYy9D
aGFuZ2VMb2cgYi9Tb3VyY2UvYm1hbGxvYy9DaGFuZ2VMb2cKaW5kZXggYzVhN2JjOTFjZjExNmQ0
YWE5Y2M1YTAyYzI5ODA5NWEzMTJhNjNmNC4uZTI1YzExYmVkZmYwM2RlN2M5NzJiOGQxMzUyZDI0
YjA1ZDY0NzliYyAxMDA2NDQKLS0tIGEvU291cmNlL2JtYWxsb2MvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9ibWFsbG9jL0NoYW5nZUxvZwpAQCAtMSw1ICsxLDE0IEBACiAyMDE4LTAyLTA5ICBDYXJs
b3MgQWxiZXJ0byBMb3BleiBQZXJleiAgPGNsb3BlekBpZ2FsaWEuY29tPgogCisgICAgICAgIElt
cHJvdmUgb2Ygc3RyaW5nLmggaW5jbHVkZSBhZnRlciByMjI4MzE3LgorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTgyNjQyCisKKyAgICAgICAgUmV2aWV3
ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgKiBibWFsbG9jL0FsZ29yaXRobS5oOiBB
dm9pZCBhbiBhcmNoaXRlY3R1cmUtc3BlY2lmaWMgI2luY2x1ZGUuCisKKzIwMTgtMDItMDkgIENh
cmxvcyBBbGJlcnRvIExvcGV6IFBlcmV6ICA8Y2xvcGV6QGlnYWxpYS5jb20+CisKICAgICAgICAg
Rml4IGJ1aWxkIGZvciAhQkNQVShYODZfNjQpIGFmdGVyIHIyMjgzMDYKICAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTE4MjU2MwogCmRpZmYgLS1naXQgYS9T
b3VyY2UvYm1hbGxvYy9ibWFsbG9jL0FsZ29yaXRobS5oIGIvU291cmNlL2JtYWxsb2MvYm1hbGxv
Yy9BbGdvcml0aG0uaAppbmRleCA4NmE1ZjkyYmFhY2JjZmQ4ZDAyNzkwMGVlNDZmNjg2OTA1YzEz
MmFkLi43OTQ4ZjQzMmFmMjZlZTVmNGI1NDBlYmJmODE1NDVkOTI0NzY0NWQyIDEwMDY0NAotLS0g
YS9Tb3VyY2UvYm1hbGxvYy9ibWFsbG9jL0FsZ29yaXRobS5oCisrKyBiL1NvdXJjZS9ibWFsbG9j
L2JtYWxsb2MvQWxnb3JpdGhtLmgKQEAgLTMxLDEzICszMSwxMCBAQAogI2luY2x1ZGUgPGNzdGRp
bnQ+CiAjaW5jbHVkZSA8Y3N0ZGRlZj4KICNpbmNsdWRlIDxsaW1pdHM+CisjaW5jbHVkZSA8c3Ry
aW5nLmg+CiAjaW5jbHVkZSA8dHlwZV90cmFpdHM+CiAjaW5jbHVkZSA8Y2hyb25vPgogCi0jaWYg
IUJDUFUoWDg2XzY0KQotI2luY2x1ZGUgPHN0cmluZy5oPgotI2VuZGlmCi0KIG5hbWVzcGFjZSBi
bWFsbG9jIHsKIAogLy8gVmVyc2lvbnMgb2YgbWluIGFuZCBtYXggdGhhdCBhcmUgY29tcGF0aWJs
ZSB3aXRoIGNvbXBpbGUtdGltZSBjb25zdGFudHMuCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>