[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-video-cvs
Subject:    [Video-cvs] sitelib/platform/win sitetext.cpp,1.7,1.8
From:       ehodge () helixcommunity ! org
Date:       2006-04-05 18:03:24
[Download RAW message or body]

Update of /cvsroot/video/sitelib/platform/win
In directory cvs02.internal.helixcommunity.org:/tmp/cvs-serv29177/platform/win

Modified Files:
	sitetext.cpp 
Log Message:
At about 40 seconds into playback, a timer causes
hxmedpltfm.dll's Plugin2Handler::UnloadDeadDLLs()
to get called to get the plugin2Handler to see if
there are any DLLs that are loaded that have a
zero use count.  If it finds any, it unloads each.
Before the proposed changes in the below diff,
vidsite.dll's base counting object's m_lNumObjects
was 0 when this happened and it thus got unloaded
at around 40 seconds.  This caused a crash
accessing a no-longer-valid memory space whenever
any site user tried to do anything with a site
thereafter.

Solution here is to have all classes that
implement an interface in video/sitelib inherit
from CHXBaseCountingObject.  That way, the base
counting object's count goes up for users of those
vidsite.dll objects (IHXSite, ...etc.)

Reviewed by: nhart@real.com


Index: sitetext.cpp
===================================================================
RCS file: /cvsroot/video/sitelib/platform/win/sitetext.cpp,v
retrieving revision 1.7
retrieving revision 1.8
diff -u -d -r1.7 -r1.8
--- sitetext.cpp	11 Mar 2005 19:58:07 -0000	1.7
+++ sitetext.cpp	5 Apr 2006 18:03:22 -0000	1.8
@@ -72,6 +72,7 @@
 #include "hxengin.h"
 #include "hxvctrl.h"
 #include "hxvsurf.h"
+#include "baseobj.h"
 #include "surface.h"
 #include "vidosurf.h"
 #ifdef _WIN32



[prev in list] [next in list] [prev in thread] [next in thread]