diff options
author | Simei Yin | 2017-07-13 23:26:47 +0200 |
---|---|---|
committer | Simei Yin | 2017-07-13 23:29:26 +0200 |
commit | b60aa5eae76c049d301ed6a6727ac2f558a4e821 (patch) | |
tree | 8f97b67228aba43c5221b58a997dec367740bfb4 /engines/sludge/builtin.cpp | |
parent | 8fde04636001fe1c902187d85302c9db2ecffec0 (diff) | |
download | scummvm-rg350-b60aa5eae76c049d301ed6a6727ac2f558a4e821.tar.gz scummvm-rg350-b60aa5eae76c049d301ed6a6727ac2f558a4e821.tar.bz2 scummvm-rg350-b60aa5eae76c049d301ed6a6727ac2f558a4e821.zip |
SLUDGE: fix warnings
Diffstat (limited to 'engines/sludge/builtin.cpp')
-rw-r--r-- | engines/sludge/builtin.cpp | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/engines/sludge/builtin.cpp b/engines/sludge/builtin.cpp index e2ebb1f16d..ee1304997c 100644 --- a/engines/sludge/builtin.cpp +++ b/engines/sludge/builtin.cpp @@ -120,7 +120,7 @@ bool failSecurityCheck(const Common::String &fn) { if (fn.empty()) return true; - for (int i = 0; i < fn.size(); ++i) { + for (uint i = 0; i < fn.size(); ++i) { switch (fn[i]) { case ':': case '\\': @@ -1077,9 +1077,8 @@ builtIn(playMovie) { builtIn(stopMovie) { UNUSEDALL - int r; - r = stopMovie(); + stopMovie(); setVariable(fun->reg, SVT_INT, 0); return BR_CONTINUE; @@ -1087,9 +1086,8 @@ builtIn(stopMovie) { builtIn(pauseMovie) { UNUSEDALL - int r; - r = pauseMovie(); + pauseMovie(); setVariable(fun->reg, SVT_INT, 0); return BR_CONTINUE; @@ -2509,7 +2507,7 @@ builtIn(setThumbnailSize) { if (!getValueType(thumbWidth, SVT_INT, fun->stack->thisVar)) return BR_ERROR; trimStack(fun->stack); - if (thumbWidth < 0 || thumbHeight < 0 || thumbWidth > winWidth || thumbHeight > winHeight) { + if (thumbWidth < 0 || thumbHeight < 0 || thumbWidth > (int)winWidth || thumbHeight > (int)winHeight) { Common::String buff = thumbWidth + " x " + thumbHeight; fatal("Invalid thumbnail size", buff); return BR_ERROR; |