Christian Semmler

Results 160 comments of Christian Semmler

`mediapath` has no function whatsoever. It's a leftover from the original decompilation. We should probably remove it from config and maybe from the INI file altogether.

Tooltips would be great - I don't think anybody is working on it right now. There's also other config related issues: https://github.com/isledecomp/isle-portable/issues/339 > (that's the different backends for the d3drm...

I wouldn't say it's expected. I'm sure there's tons of optimization potential. A modern CPU shouldn't be maxed out while still dropping below 30fps rendering LEGO Island.

Release build of course. Debug performance isn't relevant. I do get decent performance most of the time, but random frame drops occasionally. I'm mostly testing on mobile devices.

A lot of work has gone into improving performance already (thanks @AJenbo ) so I think we can close this. There's always more opportunities and other ways to improve things...

@mackron did you have time to look into this? If you could try to reproduce the issue with the sample app I've attached [here](https://github.com/isledecomp/isle-portable/pull/21#issuecomment-2156639538) that would be great. Thank you!

Tracking the issue here: https://github.com/mackron/miniaudio/issues/885

The reason is a bug internal to Miniaudio: https://github.com/mackron/miniaudio/issues/885#issuecomment-2584991710 The author knows about it, but apparently is non-trivial to fix

@AJenbo I don't think this is solved? The code still uses joystick index: https://github.com/isledecomp/isle-portable/blob/master/LEGO1/lego/legoomni/src/input/legoinputmanager.cpp#L165 And I don't think it supports more than one input device either.

Although to be fair I guess having multiple joystick inputs doesn't make much sense here, and it seems to have a falback if the index is invalid, so I guess...