Bug 27983

Summary: Explicitly include limits.h header when using INT_MAX and INT_MIN
Product: WebKit Reporter: Benjamin Meyer <ben>
Component: PlatformAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
the patch manyoso: review+

Benjamin Meyer
Reported 2009-08-04 07:45:44 PDT
Explicitly include limits.h header when using INT_MAX and INT_MIN to prevent build breakage when limits.h isn't included elsewhere by a port
Attachments
the patch (902 bytes, patch)
2009-08-04 07:46 PDT, Benjamin Meyer 		manyoso: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Benjamin Meyer
Comment 1 2009-08-04 07:46:43 PDT
Created attachment 34061 [details] the patch
Adam Treat
Comment 2 2009-08-04 07:50:46 PDT
Comment on attachment 34061 [details] the patch > From a82bc9d1cc1bb8aced54074bf3152c6f84ab560a Mon Sep 17 00:00:00 2001 > #include <stdio.h> > +#include <limits.h> > #include <wtf/Threading.h> Change the order of includes as <limit.h> is alphabetically first before <stdio.h> and then r+. You can change this when landing.
Note You need to log in before you can comment on or make changes to this bug.