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

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Oleg Romashin (:romaxa)]

I guess string.h is missing for memcpy...

Comment 1

[Oleg Romashin (:romaxa)]

Attachment: Add string.h to cover memcpy include

Comment 2

[Jeff Muizelaar [:jrmuizel]]

Comment on attachment 646616 [details] [diff] [review]
Add string.h to cover memcpy include

Please put <string.h> above the local includes

Comment 3

[Nick Cameron [:nrc]]

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

[Nick Cameron [:nrc]]

Also, I know we have problems including some of the standard libraries on mac, does that include string.h?

Comment 5

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/74c9c20b284a

Romaxa, please make sure that qnew = -U is set in your .hgrc [defaults] section.

Comment 6

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/74c9c20b284a

Comment 7

[:Ms2ger (he/him; ⌚ UTC+1/+2)]

And please address review comments...

Comment 8

[Oleg Romashin (:romaxa)]

(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)