Better handling of exceptions

abc.cpp

@@ -72,7 +72,12 @@ void DoABCTag::execute(RootMovieClip*)
 	LOG(LOG_CALLS,"ABC Exec " << Name);
 	sys->currentVm->addEvent(NULL,new ABCContextInitEvent(context));
 	SynchronizationEvent* se=new SynchronizationEvent;
-	sys->currentVm->addEvent(NULL,se);
+	bool added=sys->currentVm->addEvent(NULL,se);
+	if(!added)
+	{
+		se->decRef();
+		throw RunTimeException("Could not add event");
+	}
 	se->wait();
 	se->decRef();
 }

abc_opcodes.cpp

@@ -2094,7 +2094,7 @@ void ABCVm::newClass(call_context* th, int n)
 	//Null is a "valid" base class
 	if(tmp->getObjectType()!=T_NULL)
 	{
-		assert(tmp->getObjectType()==T_CLASS);
+		assert_and_throw(tmp->getObjectType()==T_CLASS);
 		ret->super=static_cast<Class_base*>(tmp);
 		ret->max_level=ret->super->max_level+1;
 		ret->setLevel(ret->max_level);

asobjects.cpp

@@ -17,8 +17,6 @@
     along with this program.  If not, see <http://www.gnu.org/licenses/>.
 **************************************************************************/
 
-#define __STDC_LIMIT_MACROS
-#define __STDC_CONSTANT_MACROS
 #include <list>
 #include <algorithm>
 //#include <libxml/parser.h>

exceptions.h

@@ -25,7 +25,7 @@
 
 #define assert_and_throw(cond) if(!(cond)) \
 {					\
-	throw AssertionException(##cond); \
+	throw AssertionException(#cond); \
 }
 
 

flashutils.cpp

@@ -229,7 +229,12 @@ void Timer::execute()
 			sys->currentVm->addEvent(this,Class<TimerEvent>::getInstanceS("timer"));
 			//Do not spam timer events until this is done
 			SynchronizationEvent* se=new SynchronizationEvent;
-			sys->currentVm->addEvent(NULL,se);
+			bool added=sys->currentVm->addEvent(NULL,se);
+			if(!added)
+			{
+				se->decRef();
+				throw RunTimeException("Could not add event");
+			}
 			se->wait();
 			se->decRef();
 		}

frame.cpp

@@ -93,10 +93,15 @@ void Frame::init(MovieClip* parent, list <pair<PlaceInfo, IDisplayListElem*> >&
 			if(sys->currentVm)
 			{
 				//We stop execution until execution engine catches up
-				SynchronizationEvent* s=new SynchronizationEvent;
-				sys->currentVm->addEvent(NULL, s);
-				s->wait();
-				s->decRef();
+				SynchronizationEvent* se=new SynchronizationEvent;
+				bool added=sys->currentVm->addEvent(NULL, se);
+				if(!added)
+				{
+					se->decRef();
+					throw RunTimeException("Could not add event");
+				}
+				se->wait();
+				se->decRef();
 				//Now the bindings are effective for all tags
 			}
 		}

swf.cpp

@@ -203,14 +203,18 @@ bool SystemState::shouldTerminate() const
 
 void SystemState::setError(string& c)
 {
-	error=true;
-	errorCause=c;
-	timerThread->stop();
-	if(renderThread)
+	//We record only the first error for easier fix and reporting
+	if(!error)
 	{
-		//Disable timed rendering
-		removeJob(renderThread);
-		renderThread->draw();
+		error=true;
+		errorCause=c;
+		timerThread->stop();
+		if(renderThread)
+		{
+			//Disable timed rendering
+			removeJob(renderThread);
+			renderThread->draw();
+		}
 	}
 }
 
@@ -1426,10 +1430,15 @@ void RootMovieClip::initialize()
 		//Now signal the completion for this root
 		sys->currentVm->addEvent(loaderInfo,Class<Event>::getInstanceS("init"));
 		//Wait for handling of all previous events
-		SynchronizationEvent* sync=new SynchronizationEvent;
-		sys->currentVm->addEvent(NULL, sync);
-		sync->wait();
-		sync->decRef();
+		SynchronizationEvent* se=new SynchronizationEvent;
+		bool added=sys->currentVm->addEvent(NULL, se);
+		if(!added)
+		{
+			se->decRef();
+			throw RunTimeException("Could not add event");
+		}
+		se->wait();
+		se->decRef();
 	}
 }