From 9e7d4dc49a0e8d0a06d43871dccc771a0be4074d Mon Sep 17 00:00:00 2001
From: cottonvibes <cottonvibes@96395faa-99c1-11dd-bbfe-3dabce05a288>
Date: Mon, 18 Jan 2010 09:15:21 +0000
Subject: [PATCH] newVif: fix for rule of rose.

git-svn-id: http://pcsx2.googlecode.com/svn/trunk@2452 96395faa-99c1-11dd-bbfe-3dabce05a288
---
 pcsx2/x86/newVif_Unpack.cpp | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/pcsx2/x86/newVif_Unpack.cpp b/pcsx2/x86/newVif_Unpack.cpp
index 8cf6776a03..4110f869e2 100644
--- a/pcsx2/x86/newVif_Unpack.cpp
+++ b/pcsx2/x86/newVif_Unpack.cpp
@@ -130,7 +130,7 @@ int nVifUnpack(int idx, u8* data) {
 
     if (v.partTransfer) { // Last transfer was a partial vector transfer...
 		const u8&	vifT	= nVifT[vif->cmd & 0xf];
-		const bool  doMode	= vifRegs->mode && !(vif->tag.cmd & 0x10);
+		const bool  doMode	= !!vifRegs->mode;
 		const int   diff	= vifT - v.partTransfer;
 		if (size < diff) DevCon.WriteLn("newVif: Still not enough data for unpack!");
 		memcpy(&v.partBuffer[v.partTransfer], data, diff);
@@ -142,10 +142,12 @@ int nVifUnpack(int idx, u8* data) {
     }
 
 	if (ret == v.vif->tag.size) { // Full Transfer
-		if (newVifDynaRec)  dVifUnpack(idx, data, size, isFill);
-		else			   _nVifUnpack(idx, data, size, isFill);
+		if (size > 0) {
+			if (newVifDynaRec)  dVifUnpack(idx, data, size, isFill);
+			else			   _nVifUnpack(idx, data, size, isFill);
+		}	else if (isFill)   _nVifUnpack(idx, data, size, isFill);
 		vif->tag.size = 0;
-		vif->cmd = 0; 
+		vif->cmd = 0;
 	}
 	else { // Partial Transfer
 		_nVifUnpack(idx, data, size, isFill);
@@ -227,8 +229,9 @@ __releaseinline void __fastcall _nVifUnpackLoop(u8 *data, u32 size) {
 	u8* dest			= setVUptr(idx, vuMemBase, vif->tag.addr);
 	if (vif->cl >= blockSize)  vif->cl = 0;
 	
-	while (vifRegs->num && (size >= ft.gsize)) {
+	while (vifRegs->num) {
 		if (vif->cl < cycleSize) { 
+			if (size < ft.gsize) break;
 			if (doMode) {
 				//DevCon.WriteLn("Non SSE; unpackNum = %d", upkNum);
 				func((u32*)dest, (u32*)data);
@@ -272,7 +275,7 @@ _f void _nVifUnpack(int idx, u8 *data, u32 size, bool isFill) {
 		return;
 	}
 
-	const bool doMode =  vifRegs->mode && !(vif->tag.cmd & 0x10);
+	const bool doMode = !!vifRegs->mode;
 	UnpackLoopTable[idx][doMode][isFill]( data, size );
 }
 #endif