aboutsummaryrefslogtreecommitdiff
path: root/engines
diff options
context:
space:
mode:
Diffstat (limited to 'engines')
-rw-r--r--engines/glk/adrift/os_glk.cpp58
-rw-r--r--engines/glk/adrift/scare.h3
-rw-r--r--engines/glk/adrift/scexpr.cpp2
-rw-r--r--engines/glk/adrift/sclibrar.cpp6
-rw-r--r--engines/glk/adrift/sctaffil.cpp16
-rw-r--r--engines/glk/adrift/sctafpar.cpp8
-rw-r--r--engines/glk/adrift/sxmain.cpp4
-rw-r--r--engines/glk/detection.cpp4
8 files changed, 50 insertions, 51 deletions
diff --git a/engines/glk/adrift/os_glk.cpp b/engines/glk/adrift/os_glk.cpp
index b08dfbb93a..dce48f81c6 100644
--- a/engines/glk/adrift/os_glk.cpp
+++ b/engines/glk/adrift/os_glk.cpp
@@ -82,9 +82,9 @@ static int gsc_commands_enabled = TRUE, gsc_abbreviations_enabled = TRUE,
sc_game gsc_game = nullptr;
/* Special out-of-band os_confirm() options used locally with os_glk. */
-static const sc_int GSC_CONF_SUBTLE_HINT = INT_MAX,
- GSC_CONF_UNSUBTLE_HINT = INT_MAX - 1,
- GSC_CONF_CONTINUE_HINTS = INT_MAX - 2;
+static const sc_int GSC_CONF_SUBTLE_HINT = INTEGER_MAX,
+ GSC_CONF_UNSUBTLE_HINT = INTEGER_MAX - 1,
+ GSC_CONF_CONTINUE_HINTS = INTEGER_MAX - 2;
/* Forward declaration of event wait functions, and a short delay. */
static void gsc_event_wait_2(glui32 wait_type_1,
@@ -120,7 +120,7 @@ static void gsc_fatal(const char *string) {
g_vm->glk_set_window(gsc_main_window);
g_vm->glk_set_style(style_Normal);
g_vm->glk_put_string("\n\nINTERNAL ERROR: ");
- g_vm->glk_put_string((char *) string);
+ g_vm->glk_put_string((const char *)string);
g_vm->glk_put_string("\n\nPlease record the details of this error, try to"
" note down everything you did to cause it, and email"
@@ -549,7 +549,7 @@ static void gsc_put_char_uni(glui32 unicode, const char *ascii) {
/* Print ascii to the transcript, then reattach it. */
if (gsc_transcript_stream) {
if (ascii)
- g_vm->glk_put_string_stream(gsc_transcript_stream, (char *) ascii);
+ g_vm->glk_put_string_stream(gsc_transcript_stream, (const char *)ascii);
else
g_vm->glk_put_char_stream(gsc_transcript_stream, '?');
@@ -630,7 +630,7 @@ static void gsc_put_char_locale(sc_char ch, const gsc_locale_t *locale, sc_bool
* directly with Glk.
*/
if (ascii) {
- g_vm->glk_put_string((char *) ascii);
+ g_vm->glk_put_string((const char *)ascii);
return;
}
@@ -1484,7 +1484,7 @@ static void gsc_styled_string(glui32 style, const char *message) {
assert(message);
g_vm->glk_set_style(style);
- g_vm->glk_put_string((char *) message);
+ g_vm->glk_put_string((const char *)message);
g_vm->glk_set_style(style_Normal);
}
@@ -1585,6 +1585,7 @@ void os_display_hints(sc_game game) {
* Stub functions. The unused variables defeat gcc warnings.
*/
void os_play_sound(const sc_char *filepath, sc_int offset, sc_int length, sc_bool is_looping) {
+/*
const sc_char *unused1;
sc_int unused2, unused3;
sc_bool unused4;
@@ -1592,6 +1593,7 @@ void os_play_sound(const sc_char *filepath, sc_int offset, sc_int length, sc_boo
unused2 = offset;
unused3 = length;
unused4 = is_looping;
+*/
}
void os_stop_sound() {
@@ -1640,11 +1642,13 @@ void os_show_graphic(const sc_char *filepath, sc_int offset, sc_int length) {
}
#else
void os_show_graphic(const sc_char *filepath, sc_int offset, sc_int length) {
+/*
const sc_char *unused1;
sc_int unused2, unused3;
unused1 = filepath;
unused2 = offset;
unused3 = length;
+*/
}
#endif
@@ -2527,7 +2531,7 @@ sc_bool os_confirm(sc_int type) {
do {
g_vm->glk_request_char_event(gsc_main_window);
gsc_event_wait(evtype_CharInput, &event);
- } while (event.val1 > UCHAR_MAX);
+ } while (event.val1 > BYTE_MAX);
response = g_vm->glk_char_to_upper(event.val1);
} while (response != 'Y' && response != 'N');
@@ -2601,6 +2605,9 @@ static void gsc_event_wait_2(glui32 wait_type_1, glui32 wait_type_2, event_t *ev
/* Refresh any sensitive windows on size events. */
gsc_status_redraw();
break;
+
+ default:
+ break;
}
} while (!(event->type == (EvType)wait_type_1 || event->type == (EvType)wait_type_2));
}
@@ -2656,14 +2663,14 @@ void os_write_file(void *opaque, const sc_byte *buffer, sc_int length) {
strid_t stream = (strid_t) opaque;
assert(opaque && buffer);
- g_vm->glk_put_buffer_stream(stream, (char *) buffer, length);
+ g_vm->glk_put_buffer_stream(stream, (const char *)buffer, length);
}
sc_int os_read_file(void *opaque, sc_byte *buffer, sc_int length) {
strid_t stream = (strid_t) opaque;
assert(opaque && buffer);
- return g_vm->glk_get_buffer_stream(stream, (char *) buffer, length);
+ return g_vm->glk_get_buffer_stream(stream, (char *)buffer, length);
}
@@ -2694,7 +2701,7 @@ enum gsc_end_option { GAME_RESTART, GAME_UNDO, GAME_QUIT };
* The following value needs to be passed between the startup_code and main
* functions.
*/
-static char *gsc_game_message = nullptr;
+static const char *gsc_game_message;
/*
@@ -2737,7 +2744,7 @@ static enum gsc_end_option gsc_get_ending_option() {
{
g_vm->glk_request_char_event(gsc_main_window);
gsc_event_wait(evtype_CharInput, &event);
- } while (event.val1 > UCHAR_MAX);
+ } while (event.val1 > BYTE_MAX);
response = g_vm->glk_char_to_upper(event.val1);
} while (response != 'R' && response != 'U' && response != 'Q');
@@ -2844,8 +2851,9 @@ static int gsc_startup_code(Common::SeekableReadStream *game_stream, int restore
if (!gsc_game) {
gsc_game = nullptr;
gsc_game_message = "Unable to load an Adrift game from the requested file.";
- } else
+ } else {
gsc_game_message = nullptr;
+ }
/*
* If the game was created successfully and there is a restore stream, try
@@ -3037,30 +3045,30 @@ void adrift_main() {
*/
glkunix_argumentlist_t glkunix_arguments[] = {
{
- (char *) "-nc", glkunix_arg_NoValue,
- (char *) "-nc No local handling for Glk special commands"
+ (const char *)"-nc", glkunix_arg_NoValue,
+ (const char *)"-nc No local handling for Glk special commands"
},
{
- (char *) "-na", glkunix_arg_NoValue,
- (char *) "-na Turn off abbreviation expansions"
+ (const char *)"-na", glkunix_arg_NoValue,
+ (const char *)"-na Turn off abbreviation expansions"
},
{
- (char *) "-nu", glkunix_arg_NoValue,
- (char *) "-nu Turn off any use of Unicode output"
+ (const char *)"-nu", glkunix_arg_NoValue,
+ (const char *)"-nu Turn off any use of Unicode output"
},
#ifdef LINUX_GRAPHICS
{
- (char *) "-ng", glkunix_arg_NoValue,
- (char *) "-ng Turn off attempts at game graphics"
+ (const char *)"-ng", glkunix_arg_NoValue,
+ (const char *)"-ng Turn off attempts at game graphics"
},
#endif
{
- (char *) "-r", glkunix_arg_ValueFollows,
- (char *) "-r FILE Restore from FILE on starting the game"
+ (const char *)"-r", glkunix_arg_ValueFollows,
+ (const char *)"-r FILE Restore from FILE on starting the game"
},
{
- (char *) "", glkunix_arg_ValueCanFollow,
- (char *) "filename game to run"
+ (const char *)"", glkunix_arg_ValueCanFollow,
+ (const char *)"filename game to run"
},
{nullptr, glkunix_arg_End, nullptr}
};
diff --git a/engines/glk/adrift/scare.h b/engines/glk/adrift/scare.h
index 1dc7c6e964..04a287229e 100644
--- a/engines/glk/adrift/scare.h
+++ b/engines/glk/adrift/scare.h
@@ -47,6 +47,9 @@ typedef long sc_int;
typedef unsigned long sc_uint;
typedef int sc_bool;
+#define BYTE_MAX 0xff
+#define INTEGER_MAX 0x7fff
+
/* Enumerated confirmation types, passed to os_confirm(). */
enum {
SC_CONF_QUIT = 0,
diff --git a/engines/glk/adrift/scexpr.cpp b/engines/glk/adrift/scexpr.cpp
index 0f4c2be4ce..0078f2c7ac 100644
--- a/engines/glk/adrift/scexpr.cpp
+++ b/engines/glk/adrift/scexpr.cpp
@@ -114,7 +114,7 @@ static sc_int expr_multichar_search(const sc_char *name, const sc_expr_multichar
}
/* Return the token matched, or TOK_NONE. */
- return entry->name ? entry->token : TOK_NONE;
+ return entry->name ? entry->token : (sc_int)TOK_NONE;
}
diff --git a/engines/glk/adrift/sclibrar.cpp b/engines/glk/adrift/sclibrar.cpp
index c17a3a7ae3..89f2dee7b8 100644
--- a/engines/glk/adrift/sclibrar.cpp
+++ b/engines/glk/adrift/sclibrar.cpp
@@ -1363,7 +1363,7 @@ sc_bool lib_cmd_redo_number(sc_gameref_t game) {
* for the ugliness.
*/
game->do_again = FALSE;
- game->redo_sequence = INT_MAX;
+ game->redo_sequence = INTEGER_MAX;
}
game->is_admin = TRUE;
@@ -1393,7 +1393,7 @@ static sc_bool lib_cmd_redo_text_last_common(sc_gameref_t game, const sc_char *t
/* As with failed numeric redo above, special-case this return. */
game->do_again = FALSE;
- game->redo_sequence = INT_MAX;
+ game->redo_sequence = INTEGER_MAX;
game->is_admin = TRUE;
return TRUE;
@@ -1450,7 +1450,7 @@ static sc_bool lib_cmd_redo_text_last_common(sc_gameref_t game, const sc_char *t
/* As with failed numeric redo above, special-case this return. */
game->do_again = FALSE;
- game->redo_sequence = INT_MAX;
+ game->redo_sequence = INTEGER_MAX;
}
game->is_admin = TRUE;
diff --git a/engines/glk/adrift/sctaffil.cpp b/engines/glk/adrift/sctaffil.cpp
index 321f08a192..0cda5ec20b 100644
--- a/engines/glk/adrift/sctaffil.cpp
+++ b/engines/glk/adrift/sctaffil.cpp
@@ -26,18 +26,6 @@
#include "common/zlib.h"
#include "common/memstream.h"
-#if defined(USE_ZLIB)
-#ifdef __SYMBIAN32__
-#include <zlib\zlib.h>
-#else
-#include <zlib.h>
-#endif
-
-#if ZLIB_VERNUM < 0x1204
-#error Version 1.2.0.4 or newer of zlib is required for this code
-#endif
-#endif
-
namespace Glk {
namespace Adrift {
@@ -129,7 +117,7 @@ static sc_int taf_random_state = 0x00a09e86;
static sc_byte taf_random(void) {
/* Generate and return the next pseudo-random number. */
taf_random_state = (taf_random_state * PRNG_CST1 + PRNG_CST2) & PRNG_CST3;
- return (UCHAR_MAX * (sc_uint) taf_random_state) / (sc_uint)(PRNG_CST3 + 1);
+ return (BYTE_MAX * (sc_uint) taf_random_state) / (sc_uint)(PRNG_CST3 + 1);
}
static void taf_random_reset(void) {
@@ -419,7 +407,7 @@ static sc_bool taf_unobfuscate(sc_tafref_t taf, sc_read_callbackref_t callback,
*/
static sc_bool taf_decompress(sc_tafref_t taf, sc_read_callbackref_t callback,
void *opaque, sc_bool is_gamefile) {
-#if USE_ZLIB
+#if defined(USE_ZLIB)
Common::SeekableReadStream *src = (Common::SeekableReadStream *)opaque;
assert(src);
Common::MemoryWriteStreamDynamic dest(DisposeAfterUse::YES);
diff --git a/engines/glk/adrift/sctafpar.cpp b/engines/glk/adrift/sctafpar.cpp
index 7dd7edfb4d..046cb24d33 100644
--- a/engines/glk/adrift/sctafpar.cpp
+++ b/engines/glk/adrift/sctafpar.cpp
@@ -884,14 +884,14 @@ static void parse_vector(CONTEXT, const sc_char *vector) {
}
static void parse_vector_alternate(CONTEXT, const sc_char *vector) {
- sc_int count;
+ sc_int count1;
if (parse_trace)
sc_trace("Parse: entering alternate vector %s\n", vector);
/* Element count, this is a vector described by size - 1. */
- FUNC0(parse_get_taf_integer, count) + 1;
- CALL2(parse_vector_common, vector, count);
+ FUNC0(parse_get_taf_integer, count1);
+ CALL2(parse_vector_common, vector, count1 + 1);
if (parse_trace)
sc_trace("Parse: leaving alternate vector %s\n", vector);
@@ -3240,7 +3240,7 @@ static void parse_add_alrs_index(sc_prop_setref_t bundle) {
* get the shortest and longest defined.
*/
alr_lengths = (sc_int *)sc_malloc(alr_count * sizeof(*alr_lengths));
- shortest = INT_MAX;
+ shortest = INTEGER_MAX;
longest = 0;
for (index_ = 0; index_ < alr_count; index_++) {
const sc_char *original;
diff --git a/engines/glk/adrift/sxmain.cpp b/engines/glk/adrift/sxmain.cpp
index d01e226716..63ff12ea61 100644
--- a/engines/glk/adrift/sxmain.cpp
+++ b/engines/glk/adrift/sxmain.cpp
@@ -113,13 +113,13 @@ int glk_main(int argc, const char *argv[]) {
script = sx_fopen(name, "scr", "r");
if (!script) {
- error("%s: %s.scr: %s\n", program, name, strerror(errno));
+ error("%s: %s.scr: %s\n", program, name, "open");
continue;
}
stream = sx_fopen(name, "taf", "rb");
if (!stream) {
- error("%s: %s.taf: %s\n", program, name, strerror(errno));
+ error("%s: %s.taf: %s\n", program, name, "open");
delete script;
continue;
}
diff --git a/engines/glk/detection.cpp b/engines/glk/detection.cpp
index cce1ec1ef0..4be0c38041 100644
--- a/engines/glk/detection.cpp
+++ b/engines/glk/detection.cpp
@@ -70,9 +70,9 @@ GlkDetectedGame::GlkDetectedGame(const char *id, const char *desc, const Common:
addExtraEntry("filename", filename);
}
-GlkDetectedGame::GlkDetectedGame(const char *id, const char *desc, const char *extra,
+GlkDetectedGame::GlkDetectedGame(const char *id, const char *desc, const char *xtra,
const Common::String &filename, Common::Language lang) :
- DetectedGame(id, desc, lang, Common::kPlatformUnknown, extra) {
+ DetectedGame(id, desc, lang, Common::kPlatformUnknown, xtra) {
setGUIOptions(GUIO3(GUIO_NOSPEECH, GUIO_NOMUSIC, GUIO_NOSUBTITLES));
addExtraEntry("filename", filename);
}