#2772 closed defect (fixed)
Feeble: Missing Subtitle Crash
Reported by: | SF/vhati | Owned by: | eriktorbjorn |
---|---|---|---|
Priority: | normal | Component: | Engine: AGOS |
Version: | Keywords: | ||
Cc: | Game: | The Feeble Files |
Description
2-CD English Mac Feeble Files Windows ScummVM 0.10.0svn (08/08/2006) (and 0.9.0 release)
The game completely dies (no debug popup) while talking to Gardum, hidden outside HQ.
With subtitles off, the crash doesn't happen.
When they're on, this is the last one displayed. "Distraction. Yeah, that's just what I was thinking."
I assume this is like the missing speech bug from the 6th.
Ticket imported from: #1536401. Ticket imported from: bugs/2772.
Attachments (2)
Change History (10)
by , 18 years ago
Attachment: | feeble.001 added |
---|
comment:1 by , 18 years ago
Priority: | normal → blocker |
---|
comment:2 by , 18 years ago
Changing the priority of the bug report changes nothing, this is just used by the ScummVM team, we reset user made priority changes. So please don't change it again and wait until there is news on this.
comment:3 by , 18 years ago
Priority: | blocker → normal |
---|
comment:4 by , 18 years ago
I can reproduce this crash - sometimes - but it's not that the subtitles are missing. Rather, it looks like a bug in SimonEngine::printScreenText() but, if so, one that matches the original behaviour, as far as I can tell:
If I understand it correctly, when the text is being prepared, the first line is limited to max 200 pixels. Each subsequent line is limited to the length of the line preceding it. So the lines get shorter and shorter. Eventually, the lines get so short that even one word won't fit on it.
I've attached a patch which changes the behaviour so that each line is limited to (in this case) 200 pixels, no matter what the preceding line was. That seems like more sensible behaviour to me, but I'd like to hear Kirben's opinion, before doing anything rash.
comment:5 by , 18 years ago
Owner: | set to |
---|
comment:6 by , 18 years ago
Not surprising this bug was in the original version, since printScreenText() is only used for hitbox names in the original version.
The patch looks fine, I don't recall any hitbox names that wrapped onto next time. So shouldn't be any visual difference compared to the original version.
comment:7 by , 18 years ago
Committed to both branch and trunk, after further discussion with Kirben.
comment:8 by , 18 years ago
Owner: | changed from | to
---|---|
Resolution: | → fixed |
Status: | new → closed |
Saved Game