diff options
author | Johannes Schickel | 2011-04-13 23:31:41 +0200 |
---|---|---|
committer | Johannes Schickel | 2011-04-13 23:33:22 +0200 |
commit | 41706cb4d9c2fe383de11bf84b73e46b9f9147f9 (patch) | |
tree | 5b55a8bc7ccdaf6e38d184795a1e171aa18d4822 | |
parent | c8b418036f6215729edfcfa373053f88fa3f2129 (diff) | |
download | scummvm-rg350-41706cb4d9c2fe383de11bf84b73e46b9f9147f9.tar.gz scummvm-rg350-41706cb4d9c2fe383de11bf84b73e46b9f9147f9.tar.bz2 scummvm-rg350-41706cb4d9c2fe383de11bf84b73e46b9f9147f9.zip |
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.
-rw-r--r-- | engines/tsage/converse.cpp | 7 |
1 files changed, 1 insertions, 6 deletions
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); } /** |