455355 - WinCE OGGZ file offset (no fstat in WinCE)

Status: VERIFIED WONTFIX

(Firefox for Android Graveyard :: General, defect)

Description

[John Wolfe (:wolfe)]

Attachment: Change to fseek to find out size of file, no stat structure in WinCE

Replace file stat structure usage with a simple fseek to find out the file size on WinCE builds, because stat structure is not defined under WinMobile.

Comment 1

[Doug Turner (:dougt)]

Comment on attachment 338681 [details] [diff] [review]
Change to fseek to find out size of file, no stat structure in WinCE

we should use the stub that provides a stat structure.  This code doesn't need to change, does it?

Comment 2

[John Wolfe (:wolfe)]

It will take a lot longer to fill out a stat structure for the simple "how big is this file?" query, and all other information will be abandoned.

The other alternative is to provide a stat structure that ONLY has the size (cbSize structure member) filled out correctly.  

I have a tendency to lean toward the former, since there is no confusion and no need to create a dummy stat structure and no need to create a fileno() function and a partial fstat() function (that only fills in the cbSize member).

Comment 3

[Doug Turner (:dougt)]

marking wont fix.  we will fix this in the shunt and support fstat()

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

sounds like this is not an issue anymore, marking as verified.