diff options
author | Max Horn | 2003-12-11 20:11:14 +0000 |
---|---|---|
committer | Max Horn | 2003-12-11 20:11:14 +0000 |
commit | 3080be97e0567efa78fbcbe8d2daa9906523eb8f (patch) | |
tree | 388d5d3f906c2859bbfa0bfc20d589204eedba7a /scumm | |
parent | cbafc3c8c05ef9f2427b9a3e35c07b58271bded0 (diff) | |
download | scummvm-rg350-3080be97e0567efa78fbcbe8d2daa9906523eb8f.tar.gz scummvm-rg350-3080be97e0567efa78fbcbe8d2daa9906523eb8f.tar.bz2 scummvm-rg350-3080be97e0567efa78fbcbe8d2daa9906523eb8f.zip |
Fix for bug #858262 (Note: casting to GUI::OptionsDialog* is wrong, it leads to incorrect vtable access
svn-id: r11587
Diffstat (limited to 'scumm')
-rw-r--r-- | scumm/dialogs.cpp | 21 |
1 files changed, 3 insertions, 18 deletions
diff --git a/scumm/dialogs.cpp b/scumm/dialogs.cpp index 1450f3f7e1..8cb63129b9 100644 --- a/scumm/dialogs.cpp +++ b/scumm/dialogs.cpp @@ -454,12 +454,7 @@ OptionsDialog::~OptionsDialog() { } void OptionsDialog::open() { - // MSVC6 FIX (Jamieson630): - // Simple notation would be GUI::OptionsDialog::open(), but - // MSVC6 has a problem with any notation that involves - // more than one set of double colons :: - // Hence the more convoluted notation. - ((GUI::OptionsDialog *) this)->open(); + GUI::OptionsDialog::open(); // update checkboxes, too subtitlesCheckbox->setState(_scumm->_noSubtitles == false); @@ -472,12 +467,7 @@ void OptionsDialog::close() { ConfMan.set("nosubtitles", !subtitlesCheckbox->getState(), _domain); } - // MSVC6 FIX (Jamieson630): - // Simple notation would be GUI::OptionsDialog::close(), but - // MSVC6 has a problem with any notation that involves - // more than one set of double colons :: - // Hence the more convoluted notation. - ((GUI::OptionsDialog *) this)->close(); + GUI::OptionsDialog::close(); // Sync the engine with the config manager @@ -508,12 +498,7 @@ void OptionsDialog::handleCommand(CommandSender *sender, uint32 cmd, uint32 data #endif break; default: - // MSVC6 FIX (Jamieson630): - // Simple notation would be GUI::OptionsDialog::handleCommand(), but - // MSVC6 has a problem with any notation that involves - // more than one set of double colons :: - // Hence the more convoluted notation. - ((GUI::OptionsDialog *) this)->handleCommand(sender, cmd, data); + GUI::OptionsDialog::handleCommand(sender, cmd, data); } } |