ScummVM

Logged In: YES
user_id=577918

Committed to both branch and trunk, after further discussion
with Kirben.

Logged In: YES
user_id=34715

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.

Logged In: YES
user_id=577918

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.

Logged In: YES
user_id=991728

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.