RESOLVED FIXED 182642
Improve of string.h include after r228317. 
https://bugs.webkit.org/show_bug.cgi?id=182642
Summary Improve of string.h include after r228317.
Carlos Alberto Lopez Perez
Reported 2018-02-09 08:53:49 PST
From https://bugs.webkit.org/show_bug.cgi?id=182563#c22 > (In reply to Carlos Alberto Lopez Perez from comment #20) > > Committed r228317: <https://trac.webkit.org/changeset/228317> > > We should avoid architecture-specific #includes whenever possible. Adding > one #include <string.h> is not going to slow down the build very much at > all; it's not worth the cost of the include guards, IMO.
Attachments
Patch (1.18 KB, patch)
2018-02-09 09:04 PST, Carlos Alberto Lopez Perez 		no flags
DetailsFormatted DiffDiff
Patch (1.25 KB, patch)
2018-02-09 09:06 PST, Carlos Alberto Lopez Perez 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Carlos Alberto Lopez Perez
Comment 1 2018-02-09 09:04:29 PST
Created attachment 333493 [details] Patch
Carlos Alberto Lopez Perez
Comment 2 2018-02-09 09:06:16 PST
Created attachment 333494 [details] Patch
Carlos Alberto Lopez Perez
Comment 3 2018-02-09 10:03:07 PST
Comment on attachment 333494 [details] Patch Clearing flags on attachment: 333494 Committed r228324: <https://trac.webkit.org/changeset/228324>
Carlos Alberto Lopez Perez
Comment 4 2018-02-09 10:03:11 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 5 2018-02-09 10:04:26 PST
<rdar://problem/37394750>
Note You need to log in before you can comment on or make changes to this bug.