[ scummvm-Patches-728483 ] EGA LOOM: More text-drawing hackery

SourceForge.net noreply at sourceforge.net
Sun Apr 27 18:56:58 CEST 2003


Patches item #728483, was opened at 2003-04-27 18:56
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=728483&group_id=37116

Category: None
Group: None
Status: Open
Resolution: None
Priority: 5
Submitted By: Torbjörn Andersson (eriktorbjorn)
Assigned to: Nobody/Anonymous (nobody)
Summary: EGA LOOM: More text-drawing hackery

Initial Comment:
This patch fixes most of the remaining text-drawing
issues I've seen in EGA Loom. Hopefully it's more
correct than my previous text hackery.

The most noticeable issues were that text was drawn in
the wrong position, and with the wrong colour. There
were two reasons for this: First of all, the actor's
talk colour was greater than 15. We fixed that
yesterday, but only when the colour was set by
decodeParseString, not when it was set by o5_actorSet().

Fixing the text position was just a matter of extending
the hack that was already there for Indy 3. I hope
someone else understands this better than I do. (I
noticed it has been extended to cover both versions of
Indy 3 now. The snapshot I based the patch on didn't.)

The next issue was that text wasn't erased properly.
I've changed string.cpp and charset.cpp so that it
should erase the correct rectangle. I think the old
code assumed that most text was drawn as one or two
lines of left-adjusted text, while most of the text in
Loom is centered.

Finally, I changed charset.cpp to add a drop-shadow to
Loom's text. I still don't know if it should do that to
all text, or if it's conditional.

The only remaining text-drawing issue I've seen is that
text doesn't mask the background. This is noticeable
the first time you examine the Great Loom.


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=418822&aid=728483&group_id=37116




More information about the Scummvm-tracker mailing list