[Scummvm-cvs-logs] CVS: scummvm/scumm gfx.cpp,1.89,1.90 gfx.h,1.15,1.16 object.cpp,1.26,1.27 script_v8.cpp,2.28,2.29 scumm.h,1.96,1.97 vars.cpp,1.10,1.11
Max Horn
fingolfin at users.sourceforge.net
Tue Dec 24 19:49:02 CET 2002
Update of /cvsroot/scummvm/scummvm/scumm
In directory sc8-pr-cvs1:/tmp/cvs-serv22918
Modified Files:
gfx.cpp gfx.h object.cpp script_v8.cpp scumm.h vars.cpp
Log Message:
added (broken) cursor support for V8; enabled bomp drawing
Index: gfx.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/gfx.cpp,v
retrieving revision 1.89
retrieving revision 1.90
diff -u -d -r1.89 -r1.90
--- gfx.cpp 24 Dec 2002 15:45:22 -0000 1.89
+++ gfx.cpp 25 Dec 2002 03:48:27 -0000 1.90
@@ -3085,12 +3085,14 @@
size = width * height;
if (size > sizeof(_grabbedCursor))
- error("useBompCursor: cursor too big");
+ error("useBompCursor: cursor too big (%d)", size);
_cursor.width = width;
_cursor.height = height;
_cursor.animate = 0;
+ // FIXME - why exactly the +10 ? Is that to account for block headers or something?
+ // Should be documented and verified that this is appropriate for V8 bomps, or not.
decompressBomp(_grabbedCursor, im + 10, width, height);
updateCursor();
@@ -3312,6 +3314,8 @@
int len, num;
byte code, color;
+
+ // FIXME - why this +8? To skip some kind of header? Is this right for V8 ?
src += 8;
do {
Index: gfx.h
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/gfx.h,v
retrieving revision 1.15
retrieving revision 1.16
diff -u -d -r1.15 -r1.16
--- gfx.h 21 Dec 2002 01:11:41 -0000 1.15
+++ gfx.h 25 Dec 2002 03:48:27 -0000 1.16
@@ -83,8 +83,16 @@
#endif
struct BompHeader { /* Bomp header */
- uint16 unk;
- uint16 width, height;
+ union {
+ struct {
+ uint16 unk;
+ uint16 width, height;
+ } GCC_PACK old;
+
+ struct {
+ uint32 width, height;
+ } GCC_PACK v8;
+ } GCC_PACK;
} GCC_PACK;
#if !defined(__GNUC__)
Index: object.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/object.cpp,v
retrieving revision 1.26
retrieving revision 1.27
diff -u -d -r1.26 -r1.27
--- object.cpp 24 Dec 2002 16:10:29 -0000 1.26
+++ object.cpp 25 Dec 2002 03:48:27 -0000 1.27
@@ -884,6 +884,9 @@
if (numobj > _numLocalObjects)
error("findObjectInRoom: More (%d) than %d objects in room %d", numobj, _numLocalObjects, room);
+ if (_features & GF_AFTER_V7) {
+ roomptr = getResourceAddress(rtRoomScripts, room);
+ }
if (findWhat & foCodeHeader) {
searchptr = roomptr;
for (i = 0;;) {
@@ -920,6 +923,7 @@
}
}
+ roomptr = fo->roomptr;
if (findWhat & foImageHeader) {
searchptr = roomptr;
for (i = 0;;) {
@@ -1169,15 +1173,38 @@
h = READ_LE_UINT16(&foir.cdhd->v6.h) >> 3;
}
- dataptr = findResource(IMxx_tags[imgindex], foir.obim);
- if (dataptr == NULL)
- error("setCursorImg: No such image");
+ // TODO - for V8 don't use IMxx_tags. Rather, we do something similiar to the V8
+ // code in drawBlastObject. It would be *much* nicer if we could aggregate this
+ // common code into some helper functions, instead of having long convuluted
+ // cases scattered all over the place.
+ if (_features & GF_AFTER_V8) {
+ dataptr = findResource(MKID('IMAG'), foir.obim);
+ assert(dataptr);
- size = READ_BE_UINT32_UNALIGNED(dataptr + 4);
- if (size > sizeof(_grabbedCursor))
- error("setCursorImg: Cursor image too large");
+ dataptr = findResource(MKID('WRAP'), dataptr);
+ assert(dataptr);
- if ((bomp = findResource(MKID('BOMP'), dataptr)) != NULL)
+ dataptr = findResource(MKID('OFFS'), dataptr);
+ assert(dataptr);
+
+ dataptr += READ_LE_UINT32(dataptr + 4 + 4*imgindex);
+ // TODO - distinguish between SMAP and BOMP here?
+
+ // HACK - adjust dataptr here until bomp code gets adjusted for V8
+ bomp = dataptr + 2;
+ } else {
+ dataptr = findResource(IMxx_tags[imgindex], foir.obim);
+ if (dataptr == NULL)
+ error("setCursorImg: No such image");
+
+ size = READ_BE_UINT32_UNALIGNED(dataptr + 4);
+ if (size > sizeof(_grabbedCursor))
+ error("setCursorImg: Cursor image too large");
+
+ bomp = findResource(MKID('BOMP'), dataptr);
+ }
+
+ if (bomp != NULL)
useBompCursor(bomp, w, h);
else
useIm01Cursor(dataptr, w, h);
@@ -1270,23 +1297,53 @@
if (!ptr)
error("BlastObject object %d image not found", eo->number);
- img = findResource(IMxx_tags[eo->image], ptr);
- if (!img)
- img = findResource(IMxx_tags[1], ptr); // Backward compatibility with samnmax blast objects
- if (!img)
- error("blast-object %d invalid image %d (1-x)", eo->number, eo->image);
+ if (_features & GF_AFTER_V8) {
+ // The OBIM contains an IMAG, which in turn contains a WRAP, which contains
+ // an OFFS chunk and multiple BOMP chunks. To find the right BOMP, we can
+ // either use the offsets in the OFFS chunk, or iterate over all BOMPs we find.
+ // Here we use the first method.
+ img = findResource(MKID('IMAG'), ptr);
+ assert(img);
+
+ img = findResource(MKID('WRAP'), img);
+ assert(img);
+
+ img = findResource(MKID('OFFS'), img);
+ assert(img);
+
+ bomp = img + READ_LE_UINT32(img + 4 + 4*eo->image) + 8;
+ } else {
+ img = findResource(IMxx_tags[eo->image], ptr);
+ if (!img)
+ img = findResource(IMxx_tags[1], ptr); // Backward compatibility with samnmax blast objects
+
+ if (!img)
+ error("blast-object %d invalid image %d (1-x)", eo->number, eo->image);
+
+ bomp = findResourceData(MKID('BOMP'), img);
+ }
- bomp = findResourceData(MKID('BOMP'), img);
if (!bomp)
error("object %d is not a blast object", eo->number);
- bdd.srcwidth = READ_LE_UINT16(&((BompHeader *)bomp)->width);
- bdd.srcheight = READ_LE_UINT16(&((BompHeader *)bomp)->height);
+ hexdump(bomp,32);
+ if (_features & GF_AFTER_V8) {
+ bdd.srcwidth = READ_LE_UINT32(&((BompHeader *)bomp)->v8.width);
+ bdd.srcheight = READ_LE_UINT32(&((BompHeader *)bomp)->v8.height);
+ } else {
+ bdd.srcwidth = READ_LE_UINT16(&((BompHeader *)bomp)->old.width);
+ bdd.srcheight = READ_LE_UINT16(&((BompHeader *)bomp)->old.height);
+ }
+
bdd.out = vs->screenPtr + vs->xstart;
bdd.outwidth = vs->width;
bdd.outheight = vs->height;
- bdd.dataptr = bomp + 10;
+ if (_features & GF_AFTER_V8) {
+ bdd.dataptr = bomp + 8; // Why this? See also useBompCursor
+ } else {
+ bdd.dataptr = bomp + 10; // Why this? See also useBompCursor
+ }
bdd.x = eo->posX;
bdd.y = eo->posY;
bdd.scale_x = (byte)eo->scaleX;
Index: script_v8.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/script_v8.cpp,v
retrieving revision 2.28
retrieving revision 2.29
diff -u -d -r2.28 -r2.29
--- script_v8.cpp 25 Dec 2002 02:12:12 -0000 2.28
+++ script_v8.cpp 25 Dec 2002 03:48:27 -0000 2.29
@@ -513,7 +513,7 @@
byte buffer[1024];
_msgPtrToAdd = buffer;
_scriptPointer = _messagePtr = addMessageToStack(_messagePtr);
- printf("Message(%d): '%s'\n", m, buffer);
+// printf("Message(%d): '%s'\n", m, buffer);
break;
case 0xD2: // SO_PRINT_WRAP Set print wordwrap
error("decodeParseString: SO_PRINT_MUMBLE");
@@ -714,10 +714,12 @@
break;
case 0xE4: // SO_CURSOR_IMAGE Set cursor image
{
- int room, obj = popRoomAndObj(&room);
+ int idx = pop();
+ int room, obj;
+ obj = popRoomAndObj(&room);
// FIXME
- printf("setCursorImg(%d, %d, 1)\n", obj, room);
-// setCursorImg(obj, room, 1);
+ printf("setCursorImg(%d, %d, %d)\n", obj, room, idx);
+ setCursorImg(obj, room, idx);
}
break;
case 0xE5: // SO_CURSOR_HOTSPOT Set cursor hotspot
@@ -1141,6 +1143,13 @@
case 108:
// warning("o8_kludge: PaletteBuildRedirection(%d, %d, %d, %d, %d, %d)", args[1], args[2], args[3], args[4], args[5], args[6]);
break;
+ case 118:
+ enqueueObject(args[1], args[2], args[3], args[4], args[5], args[6], args[7], args[8], 3);
+ break;
+ case 119:
+ enqueueObject(args[1], args[2], args[3], args[4], args[5], args[6], args[7], args[8], 0);
+ break;
+
case 12:
case 13:
case 14:
@@ -1158,8 +1167,6 @@
case 33:
case 34:
case 109:
- case 118:
- case 119:
default:
warning("o8_kludge: default case %d", args[0]);
}
@@ -1187,7 +1194,7 @@
default:
// FIXME - hack!
push(0);
- warning("o8_kludge2: default case %d", args[0]);
+// warning("o8_kludge2: default case %d", args[0]);
}
}
Index: scumm.h
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/scumm.h,v
retrieving revision 1.96
retrieving revision 1.97
diff -u -d -r1.96 -r1.97
--- scumm.h 24 Dec 2002 16:10:31 -0000 1.96
+++ scumm.h 25 Dec 2002 03:48:27 -0000 1.97
@@ -712,7 +712,7 @@
byte animate, animateIndex;
int8 state;
} _cursor;
- byte _grabbedCursor[2048];
+ byte _grabbedCursor[8192];
byte _currentCursor;
byte _newEffect, _switchRoomEffect2, _switchRoomEffect;
Index: vars.cpp
===================================================================
RCS file: /cvsroot/scummvm/scummvm/scumm/vars.cpp,v
retrieving revision 1.10
retrieving revision 1.11
diff -u -d -r1.10 -r1.11
--- vars.cpp 24 Dec 2002 15:27:27 -0000 1.10
+++ vars.cpp 25 Dec 2002 03:48:27 -0000 1.11
@@ -247,4 +247,7 @@
VAR_CAMERA_SPEED_Y = 108;
VAR_CAMERA_ACCEL_X = 109;
VAR_CAMERA_ACCEL_Y = 110;
+
+ // var 266, 290 and 301 have something to do with cursor images, since various scripts
+ // have setCursorImg(var290,2) or setCursorImg(var266,1) or setCursorImg(var301,1)
}
More information about the Scummvm-git-logs
mailing list