From 41706cb4d9c2fe383de11bf84b73e46b9f9147f9 Mon Sep 17 00:00:00 2001
From: Johannes Schickel
Date: Wed, 13 Apr 2011 23:31:41 +0200
Subject: TSAGE: Silence uninitialized variable usage warning.

Instead of passing an uninitialized dummy va_list to Action::attached we just
pass the va parameter from SequenceManager::attached to it now, since that
parameter is not used in Action::attached, it is safe to do that.
---
 engines/tsage/converse.cpp | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

(limited to 'engines')

diff --git a/engines/tsage/converse.cpp b/engines/tsage/converse.cpp
index 83989879dc..bcda6dbdcd 100644
--- a/engines/tsage/converse.cpp
+++ b/engines/tsage/converse.cpp
@@ -320,12 +320,7 @@ void SequenceManager::attached(EventHandler *newOwner, EventHandler *fmt, va_lis
 	}
 
 	setup();
-	// TODO: This is not particulary nice, since dummy is uninitialized.
-	// Since the default Action implementation does not access the va_list
-	// parameter it should be fine though. Still it would be nice to find
-	// a better solution to this.
-	va_list dummy;
-	Action::attached(newOwner, fmt, dummy);
+	Action::attached(newOwner, fmt, va);
 }
 
 /**
-- 
cgit v1.2.3