The default bug view has changed. See this FAQ.

DrawTargetCairo.cpp:764: error: 'memcpy' was not declared in this scope

RESOLVED FIXED in mozilla17

Status

()

Core
Graphics
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: romaxa, Assigned: romaxa)

Tracking

Trunk
mozilla17
x86_64
Linux
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
I guess string.h is missing for memcpy...
(Assignee)

Comment 1

5 years ago
Created attachment 646616 [details] [diff] [review]
Add string.h to cover memcpy include
Assignee: nobody → romaxa
Attachment #646616 - Flags: review?(ncameron)
(Assignee)

Updated

5 years ago
Attachment #646616 - Flags: review?(ncameron) → review?(jmuizelaar)
Comment on attachment 646616 [details] [diff] [review]
Add string.h to cover memcpy include

Please put <string.h> above the local includes
Attachment #646616 - Flags: review?(jmuizelaar) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed

Comment 3

5 years ago
Thanks for fixing this. Could you give more details about the error, in particular which platform it showed up on? And why this wasn't caught by our build bots? Thanks!

Comment 4

5 years ago
Also, I know we have problems including some of the standard libraries on mac, does that include string.h?
https://hg.mozilla.org/integration/mozilla-inbound/rev/74c9c20b284a

Romaxa, please make sure that qnew = -U is set in your .hgrc [defaults] section.
Flags: in-testsuite-
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/74c9c20b284a
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
And please address review comments...
(Assignee)

Comment 8

5 years ago
(In reply to Nick Cameron [:nrc] from comment #3)
> Thanks for fixing this. Could you give more details about the error, in
> particular which platform it showed up on? And why this wasn't caught by our
> build bots? Thanks!

For me it fails to build on Maemo/Meego and ubuntu arm oneiric (linux)
You need to log in before you can comment on or make changes to this bug.