SMUSH: The new smush player is (still) crashing
|Reported by:||eriktorbjorn||Owned by:||fingolfin|
Even with Fingolfin's mutex changes on March 19, the smush player is still crashing because it's trying to blit to a locked surface. I think it's because waitForTimer(), which is not protected by the mutex, will call update_screen() if ScummVM receives mouse movement events.
But should it really have to do that if the mouse cursor is hidden? It seems unnecessary. Though if we rely on that, we have to make damn sure the cursor *is* hidden during smush playback. At the moment that's not guaranteed. Except for the intro movie, it was visible during smush movies when I played CMI a few days ago.
Can it even be guaranteed in the case of Full Throttle's action sequences? I haven't seen what those should really look like.
Ticket imported from: #706754. Ticket imported from: bugs/723.