Closed Bug 456644 Opened 11 years ago Closed 11 years ago

WinCE gfx cairo _cairo_win32_tmpfile does not compile


(Core :: Graphics, defect)

Windows Mobile 6 Professional
Not set





(Reporter: wolfe, Assigned: dougt)



(Keywords: mobile)


(1 file, 3 obsolete files)

There is no _open_osfhandle function available in WinCE - but there is a CRT function to return a FILE * to a temporary file - tmpfile()
Attachment #340018 - Flags: review?(pavlov)
Assignee: nobody → wolfe
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 #340018 - Attachment is obsolete: true
Attachment #340047 - Flags: review?(pavlov)
Attachment #340018 - Flags: review?(pavlov)
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()
Attached patch patch v.1 (obsolete) — Splinter Review
Assignee: wolfe → doug.turner
Attachment #340047 - Attachment is obsolete: true
Attachment #340436 - Flags: review?
Attachment #340047 - Flags: review?(vladimir)
Attachment #340436 - Attachment is patch: true
Attachment #340436 - Attachment mime type: application/octet-stream → text/plain
Attachment #340436 - Flags: review? → review?(vladimir)
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+
Attached patch patch v.2Splinter 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)
be1ce0c38423	Doug Turner — Bug 456644 - WinCE gfx cairo _cairo_win32_tmpfile does not compile. r=vlad

thanks vlad.
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.