Summary: | [Qt][WK2] Qt layer should be hardened against C API versioning in the sense of build failures | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Balazs Kelemen <kbalazs> | ||||||
Component: | WebKit2 | Assignee: | Nobody <webkit-unassigned> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | jturcotte, webkit.review.bot | ||||||
Priority: | P3 | Keywords: | Qt, QtTriaged | ||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Unspecified | ||||||||
OS: | Unspecified | ||||||||
Attachments: |
|
Description
Balazs Kelemen
2011-08-28 12:29:25 PDT
Created attachment 106408 [details]
proposed fix
Comment on attachment 106408 [details]
proposed fix
It's not correct. Since we always build against trunk we should use the current version of the client structures. This way we can notice functional changes that affect us. Another argument is that WebKit::APIClient except the version number and the layout of the structure is matching.
Created attachment 106555 [details]
fix v2
Comment on attachment 106555 [details]
fix v2
LGTM.
Comment on attachment 106555 [details] fix v2 Clearing flags on attachment: 106555 Committed r94937: <http://trac.webkit.org/changeset/94937> All reviewed patches have been landed. Closing bug. |