Changes between Initial Version and Version 2 of Ticket #1604


Ignore:
Timestamp:
Aug 20, 2012, 1:10:31 AM (12 years ago)
Author:
historic_bruno
Comment:

Philip says the VFS supported streaming at one time but it has since been removed, and rather than worrying about that, it's better to load the whole buffer into memory using something like Vfs::LoadFile().

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #1604 – Description

    initial v2  
    1 Found this error while testing a 10.5 bundle but it also occurs on Windows, this will break releases. At a guess it's from using `fopen()` which is naughty - it won't know how to read a .zip file, I think we should be using something like `Vfs::LoadFile()` instead (this implies using a play mode that is compatible with archives). Also it would be nicer to pass an `OsPath` rather than `wchar_t*` to `InitOggFile()`. See the old [source:/ps/trunk/source/lib/res/sound/ogg.cpp?rev=11209 ogg.cpp] for ideas.
     1Found this error while testing a 10.5 bundle but it also occurs on Windows, this will break releases. At a guess it's from using `fopen()` which is naughty - it won't know how to read a .zip file, I think we should be using something like `Vfs::LoadFile()` instead (this implies using a loading method that is compatible with archives). Also it would be nicer to pass an `OsPath` rather than `wchar_t*` to `InitOggFile()`. See the old [source:/ps/trunk/source/lib/res/sound/ogg.cpp?rev=11209 ogg.cpp] for ideas.
    22
    33gdb backtrace: