[Scummvm-git-logs] scummvm master -> 931e3d82d6ee780065b60bdeb0f2399683db47d0
djsrv
dservilla at gmail.com
Sat Aug 1 03:46:50 UTC 2020
This automated email contains information about 2 new commits which have been
pushed to the 'scummvm' repo located at https://github.com/scummvm/scummvm .
Summary:
74ef6b8c68 DIRECTOR: LINGO: Fix comparison with empty string
931e3d82d6 DIRECTOR: LINGO: Fix copy/paste error
Commit: 74ef6b8c68167f2dae4668438fba95d54fe8ed05
https://github.com/scummvm/scummvm/commit/74ef6b8c68167f2dae4668438fba95d54fe8ed05
Author: djsrv (dservilla at gmail.com)
Date: 2020-07-31T23:44:56-04:00
Commit Message:
DIRECTOR: LINGO: Fix comparison with empty string
It was being cast to a float.
Changed paths:
engines/director/lingo/lingo.cpp
diff --git a/engines/director/lingo/lingo.cpp b/engines/director/lingo/lingo.cpp
index e9918d5af1..2a47f1f2c5 100644
--- a/engines/director/lingo/lingo.cpp
+++ b/engines/director/lingo/lingo.cpp
@@ -671,18 +671,22 @@ int Lingo::getAlignedType(const Datum &d1, const Datum &d2) {
if (d1Type == STRING || d1Type == FIELDREF) {
Common::String src = d1.asString();
- char *endPtr = 0;
- strtod(src.c_str(), &endPtr);
- if (*endPtr == 0) {
- d1Type = FLOAT;
+ if (!src.empty()) {
+ char *endPtr = 0;
+ strtod(src.c_str(), &endPtr);
+ if (*endPtr == 0) {
+ d1Type = FLOAT;
+ }
}
}
if (d2Type == STRING || d2Type == FIELDREF) {
Common::String src = d1.asString();
- char *endPtr = 0;
- strtod(src.c_str(), &endPtr);
- if (*endPtr == 0) {
- d2Type = FLOAT;
+ if (!src.empty()) {
+ char *endPtr = 0;
+ strtod(src.c_str(), &endPtr);
+ if (*endPtr == 0) {
+ d2Type = FLOAT;
+ }
}
}
Commit: 931e3d82d6ee780065b60bdeb0f2399683db47d0
https://github.com/scummvm/scummvm/commit/931e3d82d6ee780065b60bdeb0f2399683db47d0
Author: djsrv (dservilla at gmail.com)
Date: 2020-07-31T23:44:56-04:00
Commit Message:
DIRECTOR: LINGO: Fix copy/paste error
Changed paths:
engines/director/lingo/lingo.cpp
diff --git a/engines/director/lingo/lingo.cpp b/engines/director/lingo/lingo.cpp
index 2a47f1f2c5..3a218b0f33 100644
--- a/engines/director/lingo/lingo.cpp
+++ b/engines/director/lingo/lingo.cpp
@@ -680,7 +680,7 @@ int Lingo::getAlignedType(const Datum &d1, const Datum &d2) {
}
}
if (d2Type == STRING || d2Type == FIELDREF) {
- Common::String src = d1.asString();
+ Common::String src = d2.asString();
if (!src.empty()) {
char *endPtr = 0;
strtod(src.c_str(), &endPtr);
More information about the Scummvm-git-logs
mailing list