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

thebluegr at users.sourceforge.net thebluegr at users.sourceforge.net
Tue May 12 09:21:44 CEST 2009


Revision: 40472
          http://scummvm.svn.sourceforge.net/scummvm/?rev=40472&view=rev
Author:   thebluegr
Date:     2009-05-12 07:21:44 +0000 (Tue, 12 May 2009)

Log Message:
-----------
Replaced sci_hexdump() with Common::hexdump()

Modified Paths:
--------------
    scummvm/trunk/engines/sci/engine/script.cpp
    scummvm/trunk/engines/sci/engine/scriptconsole.cpp
    scummvm/trunk/engines/sci/engine/scriptdebug.cpp
    scummvm/trunk/engines/sci/scicore/sciconsole.h

Modified: scummvm/trunk/engines/sci/engine/script.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/script.cpp	2009-05-12 07:19:49 UTC (rev 40471)
+++ scummvm/trunk/engines/sci/engine/script.cpp	2009-05-12 07:21:44 UTC (rev 40472)
@@ -212,32 +212,6 @@
 	FIND_SELECTOR(syncTime);
 }
 
-int sci_hexdump(byte *data, int length, int offsetplus) {
-	char tempstr[40];
-	int i;
-
-	for (i = 0; i < length; i += 8) {
-		int j;
-
-		sprintf(tempstr, "%04x:                                 ", i + offsetplus);
-		for (j = 0; j < MIN(8, length - i); j++)
-			sprintf(tempstr + 6 + (j*3) + (j > 3), "%02x  ", data[i+j]);
-		for (j = 0; j < MIN(8, length - i); j++) {
-			int thechar;
-			thechar = data[i+j];
-			sprintf(tempstr + 31 + j, "%c", ((thechar < ' ') || (thechar > 127)) ? '.' : thechar);
-		}
-
-		for (j = 0; j < 38; j++)
-			if (!tempstr[j])
-				tempstr[j] = ' '; // get rid of sprintf's \0s
-
-		sciprintf("%s\n", tempstr);
-	}
-
-	return 0;
-}
-
 static void script_dump_object(char *data, int seeker, int objsize, const Common::StringList &selectorNames) {
 	int selectors, overloads, selectorsize;
 	int species = (int16)READ_LE_UINT16((unsigned char *) data + 8 + seeker);
@@ -247,7 +221,7 @@
 
 	sciprintf("Object\n");
 
-	sci_hexdump((unsigned char *) data + seeker, objsize - 4, seeker);
+	Common::hexdump((unsigned char *) data + seeker, objsize - 4, 16, seeker);
 	//-4 because the size includes the two-word header
 
 	sciprintf("Name: %s\n", namepos ? ((char *)(data + namepos)) : "<unknown>");
@@ -289,7 +263,7 @@
 
 	sciprintf("Class\n");
 
-	sci_hexdump((unsigned char *) data + seeker, objsize - 4, seeker);
+	Common::hexdump((unsigned char *) data + seeker, objsize - 4, 16, seeker);
 
 	sciprintf("Name: %s\n", namepos ? ((char *)data + namepos) : "<unknown>");
 	sciprintf("Superclass: %x\n", superclass);
@@ -370,19 +344,19 @@
 
 		case sci_obj_code: {
 			sciprintf("Code\n");
-			sci_hexdump(script->data + seeker, objsize - 4, seeker);
+			Common::hexdump(script->data + seeker, objsize - 4, 16, seeker);
 		};
 		break;
 
 		case 3: {
 			sciprintf("<unknown>\n");
-			sci_hexdump(script->data + seeker, objsize - 4, seeker);
+			Common::hexdump(script->data + seeker, objsize - 4, 16, seeker);
 		};
 		break;
 
 		case sci_obj_said: {
 			sciprintf("Said\n");
-			sci_hexdump(script->data + seeker, objsize - 4, seeker);
+			Common::hexdump(script->data + seeker, objsize - 4, 16, seeker);
 
 			sciprintf("%04x: ", seeker);
 			while (seeker < _seeker) {
@@ -447,25 +421,25 @@
 
 		case sci_obj_exports: {
 			sciprintf("Exports\n");
-			sci_hexdump((unsigned char *)script->data + seeker, objsize - 4, seeker);
+			Common::hexdump((unsigned char *)script->data + seeker, objsize - 4, 16, seeker);
 		};
 		break;
 
 		case sci_obj_pointers: {
 			sciprintf("Pointers\n");
-			sci_hexdump(script->data + seeker, objsize - 4, seeker);
+			Common::hexdump(script->data + seeker, objsize - 4, 16, seeker);
 		};
 		break;
 
 		case 9: {
 			sciprintf("<unknown>\n");
-			sci_hexdump(script->data + seeker, objsize - 4, seeker);
+			Common::hexdump(script->data + seeker, objsize - 4, 16, seeker);
 		};
 		break;
 
 		case sci_obj_localvars: {
 			sciprintf("Local vars\n");
-			sci_hexdump(script->data + seeker, objsize - 4, seeker);
+			Common::hexdump(script->data + seeker, objsize - 4, 16, seeker);
 		};
 		break;
 

Modified: scummvm/trunk/engines/sci/engine/scriptconsole.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/scriptconsole.cpp	2009-05-12 07:19:49 UTC (rev 40471)
+++ scummvm/trunk/engines/sci/engine/scriptconsole.cpp	2009-05-12 07:21:44 UTC (rev 40472)
@@ -912,7 +912,7 @@
 	else {
 		Resource *resource = s->resmgr->findResource(res, cmdParams[1].val, 0);
 		if (resource)
-			sci_hexdump(resource->data, resource->size, 0);
+			Common::hexdump(resource->data, resource->size, 16, 0);
 		else
 			sciprintf("Resource %s.%03d not found\n", cmdParams[0].str, cmdParams[1].val);
 	}

Modified: scummvm/trunk/engines/sci/engine/scriptdebug.cpp
===================================================================
--- scummvm/trunk/engines/sci/engine/scriptdebug.cpp	2009-05-12 07:19:49 UTC (rev 40471)
+++ scummvm/trunk/engines/sci/engine/scriptdebug.cpp	2009-05-12 07:21:44 UTC (rev 40472)
@@ -145,7 +145,7 @@
 				fprintf(stderr, "Track broken at %x after"
 				        " offset mod of %d\n",
 				        offset + notational_offset, offset_mod);
-				sci_hexdump(data, size, notational_offset);
+				Common::hexdump(data, size, 16, notational_offset);
 				return;
 			}
 			fprintf(stderr, "(rs %02x) ", cmd);
@@ -481,7 +481,7 @@
 		sciprintf("dynmem (%s): %d bytes\n",
 		          (*(DynMem *)mobj)._description ? (*(DynMem *)mobj)._description : "no description", (*(DynMem *)mobj)._size);
 
-		sci_hexdump((*(DynMem *)mobj)._buf, (*(DynMem *)mobj)._size, 0);
+		Common::hexdump((*(DynMem *)mobj)._buf, (*(DynMem *)mobj)._size, 16, 0);
 	}
 	break;
 
@@ -618,7 +618,7 @@
 			if (reg_end.segment != 0)
 				sciprintf("Block size less than or equal to %d\n", size);
 
-			sci_hexdump(block, size, 0);
+			Common::hexdump(block, size, 16, 0);
 		}
 		break;
 

Modified: scummvm/trunk/engines/sci/scicore/sciconsole.h
===================================================================
--- scummvm/trunk/engines/sci/scicore/sciconsole.h	2009-05-12 07:19:49 UTC (rev 40471)
+++ scummvm/trunk/engines/sci/scicore/sciconsole.h	2009-05-12 07:21:44 UTC (rev 40472)
@@ -148,9 +148,6 @@
 ** The internal list of int references is used by some of the basic commands.
 */
 
-
-int sci_hexdump(byte *data, int length, int offsetplus);
-
 } // End of namespace Sci
 
 #endif // SCI_SCICORE_SCICONSOLE_H


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