-
Notifications
You must be signed in to change notification settings - Fork 141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sndfile and CDAUDIO.WAD support needs backporting #347
Comments
wad audio file loading had been implemented, only audio scripts should be updated / tuned |
Script_GetSoundtrack doesn't allow to omit the file name in every track. And duplicating it a hundred times serve no purpose. So, it might require to modify Lua support code as well. |
for wad file using script should return only CDAUDIO.WAD and type = WAD, track number are used iside audio module (by track number audio module access to wad header, takes offset to the track and loads track) |
And this doesn't work because Script_GetSoundtrack concatenates the track path and file names which are stored in an array. You'd need to put the same file name into each array entry for this to work. And it requires the file names to be present. EDIT: Ok, I was mistaken. It's actually getTrackInfo which concatenates these names. This will do the trick:
|
Just not to forget: no any WAD file support exists in current OpenTomb master, nor any support for MS-ADPCM WAV files. There is no any streaming routines except OGG streaming, which currently doesn't work as it should (as it loads WHOLE ogg files into memory on level start). Solution 1: Brace ourselves and integrate sndfile once again. |
What do you mean? The above Lua script patch actually makes it possible to use WAD file for TR3. Also, similar patch is needed for Lua scripts to use WAV files for TR4/5. Only file names extensions and method need to be changed. I'm able to successfully use such a patch locally. It's true that there is no streaming support yet. All files are preloaded at the level start. |
I just mean that preloading soundtrack files on level start instead of streaming couldn't in any way count as "support". Sorry. It's absolutely ridiculous and wrong way to work with long audio files. |
@Lwmte Seems that streaming is working now. Should this be closed? |
This time audio streaming works itself, but it preload ALL audio data to the buffer (it is bad); Only one streaming works correctly: RPL video playing (streaming system); May be it will be more correct to create task for real streaming system (I though about moving all audio streaming format support to rpl code part) |
BTW, streams can't be stopped now except when Lara dies. This means that several streams can now be played simultaneously which wasn't intended. |
Streams stopped by next logic [type filtered]: only one type==background or type==chat may be played, type==one_shot do not stop other one_shot streams |
Yes. But EDIT: It's impossible to play the same |
Also, gone with "refactored old master".
The text was updated successfully, but these errors were encountered: