Summary: | Incorrect #if[n]def for building without DISPATCH_VM_PRESSURE | ||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Michael Saboff <msaboff> | ||||||||
Component: | WebCore Misc. | Assignee: | Michael Saboff <msaboff> | ||||||||
Status: | RESOLVED FIXED | ||||||||||
Severity: | Normal | CC: | aestes, joepeck | ||||||||
Priority: | P2 | ||||||||||
Version: | 528+ (Nightly build) | ||||||||||
Hardware: | Unspecified | ||||||||||
OS: | Unspecified | ||||||||||
Bug Depends on: | 62652 | ||||||||||
Bug Blocks: | |||||||||||
Attachments: |
|
Description
Michael Saboff
2011-06-14 10:32:59 PDT
Created attachment 97140 [details]
Proposed patch
Comment on attachment 97140 [details]
Proposed patch
Thanks!
Committed r88823: <http://trac.webkit.org/changeset/88823> DISPATCH_VM_PRESSURE is an enum, not a macro, so this check doesn't actually work and ends up breaking the build on machines that do have the system header. http://trac.webkit.org/changeset/88829 rolled this out. Reopening. Created attachment 97173 [details]
Updated patch
Created attachment 97176 [details]
Right updated patch
Comment on attachment 97176 [details]
Right updated patch
Thanks again.
Committed r88871: <http://trac.webkit.org/changeset/88871> |