[xiph-commits] r8715 - in trunk/oggdsf/src: lib/core/ogg/libOOOgg tests/testLibOOOgg

[illiminable at motherfish-iii.xiph.org]

Author: illiminable
Date: 2005-01-10 01:00:12 -0800 (Mon, 10 Jan 2005)
New Revision: 8715

Modified:
   trunk/oggdsf/src/lib/core/ogg/libOOOgg/OggPaginator.cpp
   trunk/oggdsf/src/tests/testLibOOOgg/TestPatternOggPageChecker.cpp
Log:
* Killed a big leak in the paginator.
* Killed another warning.

Modified: trunk/oggdsf/src/lib/core/ogg/libOOOgg/OggPaginator.cpp
===================================================================
--- trunk/oggdsf/src/lib/core/ogg/libOOOgg/OggPaginator.cpp	2005-01-10 07:17:10 UTC (rev 8714)
+++ trunk/oggdsf/src/lib/core/ogg/libOOOgg/OggPaginator.cpp	2005-01-10 09:00:12 UTC (rev 8715)
@@ -144,6 +144,8 @@
 	//debugLog<<"Accepting packet"<<endl;
 	addPacketToPage(inOggPacket);
 
+	delete inOggPacket;
+
 	return true;
 }
 bool OggPaginator::finishStream() 
@@ -283,6 +285,8 @@
 	}
 
 
+
+
 	//This will ensure that any header packet appears on it's own page...
 	//
 

Modified: trunk/oggdsf/src/tests/testLibOOOgg/TestPatternOggPageChecker.cpp
===================================================================
--- trunk/oggdsf/src/tests/testLibOOOgg/TestPatternOggPageChecker.cpp	2005-01-10 07:17:10 UTC (rev 8714)
+++ trunk/oggdsf/src/tests/testLibOOOgg/TestPatternOggPageChecker.cpp	2005-01-10 09:00:12 UTC (rev 8715)
@@ -12,7 +12,7 @@
 bool TestPatternOggPageChecker::acceptOggPage(OggPage* inOggPage) 
 {
 	bool locIsOK = true;
-	for (int i = 0; i < inOggPage->numPackets(); i++) {
+	for (unsigned long i = 0; i < inOggPage->numPackets(); i++) {
 		locIsOK = locIsOK && (mPacketChecker.acceptStampedOggPacket( (StampedOggPacket*)(inOggPage->getStampedPacket(i)->clone()) ) );
 	}