[Scummvm-cvs-logs] SF.net SVN: scummvm:[39212] scummvm/trunk/engines/sci

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Sun Mar 8 09:16:08 CET 2009


Revision: 39212
          http://scummvm.svn.sourceforge.net/scummvm/?rev=39212&view=rev
Author:   fingolfin
Date:     2009-03-08 08:16:08 +0000 (Sun, 08 Mar 2009)

Log Message:
-----------
SCI: replaced get_int_16 by READ_LE_UINT16, and got rid of inverse_16

Modified Paths:
--------------
    scummvm/trunk/engines/sci/gfx/gfx_resource.h
    scummvm/trunk/engines/sci/gfx/resource/sci_font.cpp
    scummvm/trunk/engines/sci/gfx/resource/sci_resmgr.cpp
    scummvm/trunk/engines/sci/gfx/resource/sci_view_0.cpp
    scummvm/trunk/engines/sci/gfx/resource/sci_view_1.cpp
    scummvm/trunk/engines/sci/scicore/vocab.cpp

Modified: scummvm/trunk/engines/sci/gfx/gfx_resource.h
===================================================================
--- scummvm/trunk/engines/sci/gfx/gfx_resource.h	2009-03-08 08:15:56 UTC (rev 39211)
+++ scummvm/trunk/engines/sci/gfx/gfx_resource.h	2009-03-08 08:16:08 UTC (rev 39212)
@@ -298,10 +298,6 @@
 ** the mode has a byte depth of more than 1.
 */
 
-static inline int get_int_16(const byte *offset) {
-	return ((int) offset[0] | (((int) offset[1]) << 8));
-}
-
 } // End of namespace Sci
 
 #endif // SCI_GFX_GFX_RESOURCE_H

Modified: scummvm/trunk/engines/sci/gfx/resource/sci_font.cpp
===================================================================
--- scummvm/trunk/engines/sci/gfx/resource/sci_font.cpp	2009-03-08 08:15:56 UTC (rev 39211)
+++ scummvm/trunk/engines/sci/gfx/resource/sci_font.cpp	2009-03-08 08:16:08 UTC (rev 39212)
@@ -77,8 +77,8 @@
 		return NULL;
 	}
 
-	font->chars_nr = chars_nr = get_int_16(resource + FONT_MAXCHAR_OFFSET);
-	font->line_height = max_height = get_int_16(resource + FONT_HEIGHT_OFFSET);
+	font->chars_nr = chars_nr = READ_LE_UINT16(resource + FONT_MAXCHAR_OFFSET);
+	font->line_height = max_height = READ_LE_UINT16(resource + FONT_HEIGHT_OFFSET);
 
 	if (chars_nr < 0 || chars_nr > 256 || max_height < 0) {
 		if (chars_nr < 0 || chars_nr > 256)
@@ -99,7 +99,7 @@
 	font->widths = (int*)sci_malloc(sizeof(int) * chars_nr);
 
 	for (i = 0; i < chars_nr; i++) {
-		int offset = get_int_16(resource + (i << 1) + 6);
+		int offset = READ_LE_UINT16(resource + (i << 1) + 6);
 
 		if (offset >= size) {
 			GFXERROR("Font %04x: Error: Character 0x%02x is at offset 0x%04x (beyond 0x%04x)\n", id, i, offset, size);
@@ -128,7 +128,7 @@
 	font->data = (byte *)sci_calloc(font->char_size, chars_nr);
 
 	for (i = 0; i < chars_nr; i++) {
-		int offset = get_int_16(resource + (i << 1) + 6);
+		int offset = READ_LE_UINT16(resource + (i << 1) + 6);
 
 		if (calc_char(font->data + (font->char_size * i), font->row_size, max_height, resource + offset, size - offset)) {
 			GFXERROR("Problem occured in font %04x, char %d/%d\n", id, i, chars_nr);

Modified: scummvm/trunk/engines/sci/gfx/resource/sci_resmgr.cpp
===================================================================
--- scummvm/trunk/engines/sci/gfx/resource/sci_resmgr.cpp	2009-03-08 08:15:56 UTC (rev 39211)
+++ scummvm/trunk/engines/sci/gfx/resource/sci_resmgr.cpp	2009-03-08 08:16:08 UTC (rev 39212)
@@ -241,8 +241,8 @@
 	retval->color_key = GFX_CURSOR_TRANSPARENT;
 
 	if (state->version != SCI_VERSION_0) {
-		retval->xoffset = get_int_16(resource);
-		retval->yoffset = get_int_16(resource + 2);
+		retval->xoffset = READ_LE_UINT16(resource);
+		retval->yoffset = READ_LE_UINT16(resource + 2);
 	} else if (resource[3]) // center
 		retval->xoffset = retval->yoffset = CURSOR_SIZE / 2;
 	else
@@ -252,8 +252,8 @@
 
 	data = retval->index_data;
 	for (line = 0; line < 16; line++) {
-		int mask_a = get_int_16(resource + (line << 1));
-		int mask_b = get_int_16(resource + 32 + (line << 1));
+		int mask_a = READ_LE_UINT16(resource + (line << 1));
+		int mask_b = READ_LE_UINT16(resource + 32 + (line << 1));
 		int i;
 
 		for (i = 0; i < 16; i++) {

Modified: scummvm/trunk/engines/sci/gfx/resource/sci_view_0.cpp
===================================================================
--- scummvm/trunk/engines/sci/gfx/resource/sci_view_0.cpp	2009-03-08 08:15:56 UTC (rev 39211)
+++ scummvm/trunk/engines/sci/gfx/resource/sci_view_0.cpp	2009-03-08 08:16:08 UTC (rev 39212)
@@ -33,8 +33,8 @@
 namespace Sci {
 
 gfx_pixmap_t *gfxr_draw_cel0(int id, int loop, int cel, byte *resource, int size, gfxr_view_t *view, int mirrored) {
-	int xl = get_int_16(resource);
-	int yl = get_int_16(resource + 2);
+	int xl = READ_LE_UINT16(resource);
+	int yl = READ_LE_UINT16(resource + 2);
 	int xhot = ((signed char *)resource)[4];
 	int yhot = ((signed char *)resource)[5];
 	int color_key = resource[6];
@@ -122,7 +122,7 @@
 
 static int gfxr_draw_loop0(gfxr_loop_t *dest, int id, int loop, byte *resource, int offset, int size, gfxr_view_t *view, int mirrored) {
 	int i;
-	int cels_nr = get_int_16(resource + offset);
+	int cels_nr = READ_LE_UINT16(resource + offset);
 
 	if (READ_LE_UINT16(resource + offset + 2)) {
 		GFXWARN("View %02x:(%d): Gray magic %04x in loop, expected white\n", id, loop, READ_LE_UINT16(resource + offset + 2));
@@ -170,7 +170,7 @@
 	gfxr_view_t *view;
 	int mirror_bitpos = 1;
 	int mirror_bytepos = V0_MIRROR_LIST_OFFSET;
-	int palette_ofs = get_int_16(resource + 6);
+	int palette_ofs = READ_LE_UINT16(resource + 6);
 
 	if (size < V0_FIRST_LOOP_OFFSET + 8) {
 		GFXERROR("Attempt to draw empty view %04x\n", id);

Modified: scummvm/trunk/engines/sci/gfx/resource/sci_view_1.cpp
===================================================================
--- scummvm/trunk/engines/sci/gfx/resource/sci_view_1.cpp	2009-03-08 08:15:56 UTC (rev 39211)
+++ scummvm/trunk/engines/sci/gfx/resource/sci_view_1.cpp	2009-03-08 08:16:08 UTC (rev 39212)
@@ -230,8 +230,8 @@
 }
 
 gfx_pixmap_t *gfxr_draw_cel1(int id, int loop, int cel, int mirrored, byte *resource, int size, gfxr_view_t *view, int amiga_game) {
-	int xl = get_int_16(resource);
-	int yl = get_int_16(resource + 2);
+	int xl = READ_LE_UINT16(resource);
+	int yl = READ_LE_UINT16(resource + 2);
 	int xhot = (int8) resource[4];
 	int yhot = (uint8) resource[5];
 	int pos = 8;
@@ -274,7 +274,7 @@
 
 static int gfxr_draw_loop1(gfxr_loop_t *dest, int id, int loop, int mirrored, byte *resource, int offset, int size, gfxr_view_t *view, int amiga_game) {
 	int i;
-	int cels_nr = get_int_16(resource + offset);
+	int cels_nr = READ_LE_UINT16(resource + offset);
 
 	if (READ_LE_UINT16(resource + offset + 2)) {
 		GFXWARN("View %02x:(%d): Gray magic %04x in loop, expected white\n", id, loop, READ_LE_UINT16(resource + offset + 2));

Modified: scummvm/trunk/engines/sci/scicore/vocab.cpp
===================================================================
--- scummvm/trunk/engines/sci/scicore/vocab.cpp	2009-03-08 08:15:56 UTC (rev 39211)
+++ scummvm/trunk/engines/sci/scicore/vocab.cpp	2009-03-08 08:16:08 UTC (rev 39212)
@@ -154,10 +154,6 @@
 	return "{invalid}";
 }
 
-static inline unsigned int inverse_16(unsigned int foo) {
-	return (((foo & 0xff) << 8) | ((foo & 0xff00) >> 8));
-}
-
 suffix_t **vocab_get_suffices(ResourceManager *resmgr, int *suffices_nr) {
 	int counter = 0;
 	suffix_t **suffices;
@@ -191,10 +187,10 @@
 
 		suffices[counter]->alt_suffix_length = alt_len;
 		suffices[counter]->word_suffix_length = word_len;
-		suffices[counter]->class_mask = inverse_16((int16)READ_LE_UINT16(resource->data + seeker)); // Inverse endianness
+		suffices[counter]->class_mask = (int16)READ_BE_UINT16(resource->data + seeker);
 
 		seeker += word_len + 4;
-		suffices[counter]->result_class = inverse_16((int16)READ_LE_UINT16(resource->data + seeker));
+		suffices[counter]->result_class = (int16)READ_BE_UINT16(resource->data + seeker);
 		seeker += 3; // Next entry
 
 		++counter;


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list