cairo-wideint-type-private.h redefines int types with MSVC10

RESOLVED INVALID

Status

()

Core
Graphics
RESOLVED INVALID
8 years ago
4 years ago

People

(Reporter: Mitch, Assigned: Mitch)

Tracking

Trunk
mozilla2.0
x86
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

8 years ago
MSVC10 has stdint.h but cairo-wideint-type-private.h defines its own types anyway. Patch upcoming.
(Assignee)

Comment 1

8 years ago
Created attachment 491528 [details] [diff] [review]
Patch

This patch uses an MSVC version check. I'm not sure why HAVE_STDINT_H isn't defined, but for reference a version check was used here:

http://hg.mozilla.org/mozilla-central/file/b433764562cc/gfx/cairo/libpixman/src/pixman.h#l94
Attachment #491528 - Flags: review?(jmuizelaar)
Comment on attachment 491528 [details] [diff] [review]
Patch

It seems like HAVE_STDINT_H should be true on VS2010 so we shouldn't have to work around it.
Attachment #491528 - Flags: review?(jmuizelaar) → review-
Status: ASSIGNED → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.