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

[wjpalenstijn at users.sourceforge.net]

Revision: 40838
          http://scummvm.svn.sourceforge.net/scummvm/?rev=40838&view=rev
Author:   wjpalenstijn
Date:     2009-05-23 22:21:42 +0000 (Sat, 23 May 2009)

Log Message:
-----------
SCI: add (unused) palette revisioning support

Modified Paths:
--------------
    scummvm/trunk/engines/sci/gfx/palette.cpp
    scummvm/trunk/engines/sci/gfx/palette.h

Modified: scummvm/trunk/engines/sci/gfx/palette.cpp
===================================================================
--- scummvm/trunk/engines/sci/gfx/palette.cpp	2009-05-23 21:48:33 UTC (rev 40837)
+++ scummvm/trunk/engines/sci/gfx/palette.cpp	2009-05-23 22:21:42 UTC (rev 40838)
@@ -36,6 +36,7 @@
 	_parent = 0;
 	_dirty = true;
 	_refcount = 1;
+	_revision = 0;
 }
 
 Palette::Palette(const gfx_pixmap_color_t *colors, uint s) {
@@ -44,6 +45,7 @@
 	_parent = 0;
 	_dirty = true;
 	_refcount = 1;
+	_revision = 0;
 
 	for (uint i = 0; i < _size; ++i) {
 		_colors[i].r = colors[i].r;
@@ -97,6 +99,12 @@
 #ifdef DEBUG_MERGE
 	fprintf(stderr, "Unmerge %s from %s (%d colors)\n", name.c_str(), _parent->name.c_str(), _size);
 #endif
+	if (_parent->_revision != _revision) {
+#ifdef DEBUG_MERGE
+		fprintf(stderr, "NOP (revision mismatch)\n");
+#endif
+		return;
+	}
 
 	int count = 0;
 	for (uint i = 0; i < _size; ++i) {
@@ -108,9 +116,6 @@
 		assert(pi < (int)_parent->_size);
 		assert(_parent->_colors[pi].refcount != 0);
 		assert(_parent->_colors[pi].refcount != PALENTRY_FREE);
-#ifdef DEBUG_MERGE
-		int old = _parent->_colors[pi].refcount;
-#endif
 		if (_parent->_colors[pi].refcount != PALENTRY_LOCKED)
 			_parent->_colors[pi].refcount--;
 		if (_parent->_colors[pi].refcount == 0) {
@@ -211,28 +216,29 @@
 	return bestcolor;
 }
 
-void Palette::mergeInto(Palette *parent) {
+bool Palette::mergeInto(Palette *parent) {
 	assert(!_parent || _parent == parent);
 	assert(parent != this);
 #ifdef DEBUG_MERGE
 	fprintf(stderr, "Merge: %s into %s (%d colors)\n", name.c_str(), parent->name.c_str(), _size);
 #endif
 
-	if (_parent == parent) {
+	if (_parent == parent && parent->_revision == _revision) {
 #ifdef DEBUG_MERGE
 		fprintf(stderr, "NOP\n");
 #endif
-		return;
+		return false;
 	}
 	_parent = parent;
+	_revision = parent->_revision;
 
 #ifdef DEBUG_MERGE
 	bool *used = new bool[_parent->size()];
 	for (uint i = 0; i < _parent->size(); ++i)
 		used[i] = false;
 	int count = 0;
-	int used_min = 1000;
-	int used_max = 0;
+	uint used_min = 1000;
+	uint used_max = 0;
 #endif
 
 	for (uint i = 0; i < _size; ++i) {
@@ -255,8 +261,35 @@
 	fprintf(stderr, "Merge used %d colours in [%d..%d]\n", count, used_min, used_max);
 	delete[] used;
 #endif
+	return true;
 }
 
+void Palette::forceInto(Palette *parent) {
+	assert(!_parent || _parent == parent);
+	assert(parent != this);
+#ifdef DEBUG_MERGE
+	fprintf(stderr, "Merge: force %s into %s (%d colors)\n", name.c_str(), parent->name.c_str(), _size);
+#endif
+
+	_parent = parent;
+	parent->_revision++;
+	_revision = parent->_revision;
+
+	for (unsigned int i = 0; i < _size; ++i) {
+		// FIXME: PALENTRY_LOCKED doesn't work well with forceInto...
+		if (_colors[i].refcount != PALENTRY_FREE) {
+			_parent->_colors[i] = _colors[i];
+			_parent->_colors[i].parent_index = -1;
+			_colors[i].parent_index = i;
+			if (_parent->_colors[i].refcount != PALENTRY_LOCKED)
+				_parent->_colors[i].refcount = 1;
+		} else {
+			_parent->_colors[i].refcount = 0;
+		}
+	}
+	_parent->_dirty = true;
+}
+
 Palette *Palette::copy() {
 	assert(!_parent);
 	Palette* p = new Palette(_size);
@@ -264,7 +297,8 @@
 
 	for (uint i = 0; i < _size; ++i) {
 		p->_colors[i] = _colors[i];
-		p->_colors[i].refcount = 0;
+		if (p->_colors[i].refcount >= 0)
+			p->_colors[i].refcount = 0;
 	}
 	return p;
 }

Modified: scummvm/trunk/engines/sci/gfx/palette.h
===================================================================
--- scummvm/trunk/engines/sci/gfx/palette.h	2009-05-23 21:48:33 UTC (rev 40837)
+++ scummvm/trunk/engines/sci/gfx/palette.h	2009-05-23 22:21:42 UTC (rev 40838)
@@ -80,12 +80,15 @@
 	bool isDirty() const { return _dirty; }
 	bool isShared() const { return _refcount > 1; }
 	Palette *getParent() { return _parent; }
+	int getRevision() const { return _revision; }
 
 	void markClean() { _dirty = false; }
 
 	uint findNearbyColor(byte r, byte g, byte b, bool lock=false);
 
-	void mergeInto(Palette *parent);
+	bool mergeInto(Palette *parent); // returns false if already merged
+	void forceInto(Palette *parent);
+
 	void unmerge();
 
 	Common::String name; // strictly for debugging purposes
@@ -97,6 +100,8 @@
 
 	bool _dirty; // Palette has changed
 	int _refcount; // Number of pixmaps (or other objects) using this palette
+	int _revision; // When this is incremented, all child references are
+	               // invalidated
 };
 
 


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