Closed Bug 456644 Opened 11 years ago Closed 11 years ago
CE gfx cairo _cairo _win32 _tmpfile does not compile
There is no _open_osfhandle function available in WinCE - but there is a CRT function to return a FILE * to a temporary file - tmpfile()
I got the _cairo_win32_tmpfile() function inside cairo-win32.c, but missed the identical function inside cairo-misc.c This updated patch does the identical change to the two identical source code functions.
Attachment #340047 - Flags: review?(pavlov) → review?(vladimir)
Comment on attachment 340047 [details] [diff] [review] Duplicated _cairo_win32_tmpfile() functions - missed one on the first pass please add this patch to gfx/cairo/ and update the README with the patch so it can be pushed upstream
Could the existing cairo code be replaced with tmpfile() on Windows altogether? Without the need to special case WINCE?
we can't -- see the comment at the top of _cairo_win32_tmpfile()
Comment on attachment 340436 [details] [diff] [review] patch v.1 Looks fine, but can you just delete the duplicate function in cairo-win32.c? No reason for it to be there, was just an omission in not removing it when it got moved to -misc.c
Attachment #340436 - Flags: review?(vladimir) → review+
after talking to vlad, this patch removes the cairo-win32.c file from the Makefile. This change needs to be reflected upstream.
Attachment #340436 - Attachment is obsolete: true
Attachment #340450 - Flags: review?(vladimir) → review+
be1ce0c38423 Doug Turner — Bug 456644 - WinCE gfx cairo _cairo_win32_tmpfile does not compile. r=vlad thanks vlad.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Component: General → GFX: Thebes
Product: Fennec → Core
QA Contact: general → thebes
You need to log in before you can comment on or make changes to this bug.