The default bug view has changed. See this FAQ.

cairoint.h uses lround in C90 programs

RESOLVED FIXED in mozilla9

Status

()

Core
Graphics
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Ehsan, Assigned: Ehsan)

Tracking

Trunk
mozilla9
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
lround is a function available in C99 programs, but cairo-int.h is using it in C90 programs as well, which causes libxul to fail to link when building in C90 mode.
(Assignee)

Comment 1

6 years ago
Created attachment 557650 [details] [diff] [review]
Patch (v1)
Assignee: nobody → ehsan
Status: NEW → ASSIGNED
Attachment #557650 - Flags: review?(jmuizelaar)
(Assignee)

Updated

6 years ago
Summary: cairo-int.h uses lround in C90 programs → cairoint.h uses lround in C90 programs
Comment on attachment 557650 [details] [diff] [review]
Patch (v1)

Add a patch to the cairo directory and a note in the README
Attachment #557650 - Flags: review?(jmuizelaar) → review+
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/4e76b1e5c1d1
Target Milestone: --- → mozilla9
(Assignee)

Comment 4

6 years ago
http://hg.mozilla.org/mozilla-central/rev/4e76b1e5c1d1
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.