#2309 closed defect (fixed)
Scumm: Crash with subtitle function
| Reported by: | SF/simsaw | Owned by: | Kirben |
|---|---|---|---|
| Priority: | normal | Component: | GUI |
| Version: | Keywords: | ||
| Cc: | Game: |
Description
When you decrease the subtitle to the lowest point and you try to increase it again (the subtitle speed display has to disappear before) ScummVM crashes. It also happens the other way round.
Ticket imported from: #1337498. Ticket imported from: bugs/2309.
Change History (11)
comment:1 by , 19 years ago
comment:2 by , 19 years ago
| Owner: | set to |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
comment:3 by , 19 years ago
Apparently, this was introduced with bug #1335405. As it's written there, the 'talk speed' dialog which is displayed when the user presses '-' or '+' is a bit misleading. With the '-' or '+' keys, the "talkDelay" gets modified. But we display it as "talkSpeed" (which is more or less the opposite).
I am not sure what is the best solution : - change the "talk speed" caption dialog by "talk delay" - or as Kirben did, swap the "_lastKeyHit" conditions - something else ?
comment:4 by , 19 years ago
I didn't consider that. All I know is that the old behaviour was completely wrong, because the first keypress would move the gauge in one direction, while subsequent ones would move it in the other direction. Which is what could lead to a failed assertion.
comment:5 by , 19 years ago
| Status: | closed → new |
|---|
comment:6 by , 19 years ago
Reopening and assigning to Kirben for further comments, since he seems to have made the change cyx mentioned.
I don't know what the correct behaviour is, but surely a gauge that increases when you press - and decreases when you press + would be extremely confusing...?
comment:7 by , 19 years ago
| Owner: | changed from to |
|---|---|
| Resolution: | fixed |
comment:8 by , 19 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:9 by , 19 years ago
We should match the keyboard shortcuts behaviour of the original games, where '-' decreased the subtitle speed and '+' increased the subtitle speed. I changed ScummVM to match the original games again, remembering to adjust the safety checks this time. I renamed 'Talk Speed' to 'Talk Delay' to avoid any confusion about the subtitle speed controls.
comment:10 by , 19 years ago
For future reference: We used to match the subtitle speed controls of the original games, as our README still states. Fingolfin changed the subtitle speed controls, when he added that GUI dialog.
comment:11 by , 6 years ago
| Component: | → GUI |
|---|
Looks like a simple mix-up: The first time you hit + the delay value would decrease, and the first time you hit - the value would increase. That's what caused the value to overflow/underflow when the speed display was opened.
I've changed it to be the other way around. Thanks for pointing it out!