Opened 14 years ago

Closed 14 years ago

Last modified 12 months ago

#8484 closed patch

KYRA: Chat and various fixes

Reported by: vinterstum Owned by: lordhoto
Priority: normal Component: Engine: Kyra
Keywords: Cc:


Implements the cmd_characterSay() opcode and numerous
supporting functions. Also does a few various fixes,
and modifies the scene animations to better mesh with
the rest of the system.

Ticket imported from: #1350510. Ticket imported from: patches/589.

Attachments (3)

kyra_chat_plus_plus.patch (36.9 KB ) - added by vinterstum 14 years ago.
KYRA: chat and sundry
kyra_chat_plus_plus_v2.patch (58.5 KB ) - added by vinterstum 14 years ago.
kyra_chat_plus_plus_v3.patch (58.8 KB ) - added by vinterstum 14 years ago.
KYRA: chat++

Download all attachments as: .zip

Change History (15)

by vinterstum, 14 years ago

Attachment: kyra_chat_plus_plus.patch added

KYRA: chat and sundry

comment:1 by lordhoto, 14 years ago

Owner: set to lordhoto

comment:2 by fingolfin, 14 years ago

LordHoto, you took this item -- now, is there any reason you didn't just
apply it? If so, please add them to this tracker item.

comment:3 by lordhoto, 14 years ago

Well I had no time but I wanted to look at it first, before
commiting anything. So maybe it will be accepted just as is.

comment:4 by fingolfin, 14 years ago

Oh no no, absolutley no reason for undue hurry :-). It's all
my fault, I thought that lordhoto had taken this item by
himself and was wondering why no message appeared -- sorry,
should have checked more carefully!. As long as I know this
item is "in progress", and that comments will be posted here
(and not just in private emails), I have absolutley no
problem waiting a couple days/weeks/months for it to be
resolved (the submitter might, but that's another thing ;-).

comment:5 by lordhoto, 14 years ago

well it was the other item, this one I took by myself...

comment:6 by lordhoto, 14 years ago

Well back to on topic now:

- there are some bugs with the drawposition of the sprites
as it looks
- also there is a force scene change that is not used, it is
only not done because out of some missing functionallity.
- also there is some debug output, maybe you should show it
only at a special debuglevel (maybe 1 or 2 or something)
- also cmd_characterSays looks strange since the strings
compared there are looking odd, like binary data

I think you should take a look at it, before it gets commited.
But nice work.

comment:7 by vinterstum, 14 years ago

(to repeat stuff from IRC, for the record :)

- The drawing issue (a width issue) is due to
Screen::copyRegion() which currently expects the width
parameter to be the actual width, and not 1/8 of the width
(as in the original engine). Some of our other code expects
it to be like that, some of it expects it to be like the
- The scene change wasn't mean to be a permanent addition :)
- The debug output was an oversight, will correct it.
- Yeah, the strings in cmd_characterSays are odd. I've
double and tripplechecked them though. As you speculated
when I originally mentioned it, they might be relevant to
the Japanese version only.

comment:8 by lordhoto, 14 years ago

Well if it is only used for the japanese version I think it
would be better to leave it out since we do not support it.

by vinterstum, 14 years ago

comment:9 by vinterstum, 14 years ago

* Fixes sprite drawing
* Hands mouse drawing back to OSystem
* Adds support for cmd_message() and supporting function
* Added various animation-related opcodes
* Added a function which does more room initialization
(without the palette stuff, which doesn't seem necessary).
* Probably more I've forgotten.

by vinterstum, 14 years ago

KYRA: chat++

comment:10 by lordhoto, 14 years ago

Commited with a few changes, see cvs logs for the changes.

comment:11 by lordhoto, 14 years ago

Status: newclosed

comment:12 by digitall, 12 months ago

Component: Engine: Kyra
Note: See TracTickets for help on using tickets.