From 84398f85d8cad3ee2805cfdc6f5563aa4c7748d6 Mon Sep 17 00:00:00 2001 From: D G Turner Date: Sat, 23 Nov 2019 12:38:11 +0000 Subject: GLK: MAGNETIC: Fix Compilation on AmigaOS4 This seems to be an issue where the initializer for the members called log get confused for a call to the log() standard library function. Renaming these members with leading underscores and adding some checks for nullptr before usage are good practice in any case and should fix this. --- engines/glk/magnetic/glk.cpp | 12 ++++++++---- engines/glk/magnetic/magnetic.cpp | 2 +- engines/glk/magnetic/magnetic.h | 2 +- 3 files changed, 10 insertions(+), 6 deletions(-) diff --git a/engines/glk/magnetic/glk.cpp b/engines/glk/magnetic/glk.cpp index 7b70fa3976..1c0a366671 100644 --- a/engines/glk/magnetic/glk.cpp +++ b/engines/glk/magnetic/glk.cpp @@ -3980,13 +3980,17 @@ void Magnetic::writeChar(char c) { } void Magnetic::script_write(type8 c) { - if (log_on == 2) - log1->writeByte(c); + if (log_on == 2) { + if (_log1) { + _log1->writeByte(c); + } + } } void Magnetic::transcript_write(type8 c) { - if (log2) - log2->writeByte(c); + if (_log2) { + _log2->writeByte(c); + } } } // End of namespace Magnetic diff --git a/engines/glk/magnetic/magnetic.cpp b/engines/glk/magnetic/magnetic.cpp index 581d246acf..9e909bc498 100644 --- a/engines/glk/magnetic/magnetic.cpp +++ b/engines/glk/magnetic/magnetic.cpp @@ -64,7 +64,7 @@ Magnetic::Magnetic(OSystem *syst, const GlkGameDescription &gameDesc) : GlkAPI(s pos_table_index(-1), pos_table_max(-1), anim_repeat(0) #endif , hints(nullptr), hint_contents(nullptr), xpos(0), bufpos(0), log_on(0), - ms_gfx_enabled(0), log1(nullptr), log2(nullptr), GMS_LUMINANCE_WEIGHTS(299, 587, 114), + ms_gfx_enabled(0), _log1(nullptr), _log2(nullptr), GMS_LUMINANCE_WEIGHTS(299, 587, 114), linear_gamma(nullptr), pic_current_crc(0), hints_current_crc(0), hints_crc_initialized(false), _saveData(nullptr), _saveSize(0) { diff --git a/engines/glk/magnetic/magnetic.h b/engines/glk/magnetic/magnetic.h index a3ea9ceee4..c19ef9c8b5 100644 --- a/engines/glk/magnetic/magnetic.h +++ b/engines/glk/magnetic/magnetic.h @@ -259,7 +259,7 @@ private: size_t _saveSize; private: type8 buffer[80], xpos, bufpos, log_on, ms_gfx_enabled, filename[256]; - Common::DumpFile *log1, *log2; + Common::DumpFile *_log1, *_log2; private: /* Method local statics in original code */ glui32 crc_table[BYTE_MAX_VAL + 1]; -- cgit v1.2.3