Remove unused variables and cleanup precision warnings as dSprintf takes a U32 for the size of the buffer to use.

This commit is contained in:
Jeff Hutchinson 2018-04-02 23:38:17 -04:00
parent 6d37709b51
commit 050d710580

View file

@ -426,12 +426,12 @@ exitLabel:
} }
if (thisNamespace && thisNamespace->mName) if (thisNamespace && thisNamespace->mName)
{ {
dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), sizeof(sTraceBuffer) - dStrlen(sTraceBuffer), dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), (U32)(sizeof(sTraceBuffer) - dStrlen(sTraceBuffer)),
"%s::%s() - return %s", thisNamespace->mName, mThisFunctionName, STR.getStringValue()); "%s::%s() - return %s", thisNamespace->mName, mThisFunctionName, STR.getStringValue());
} }
else else
{ {
dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), sizeof(sTraceBuffer) - dStrlen(sTraceBuffer), dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), (U32)(sizeof(sTraceBuffer) - dStrlen(sTraceBuffer)),
"%s() - return %s", mThisFunctionName, STR.getStringValue()); "%s() - return %s", mThisFunctionName, STR.getStringValue());
} }
Con::printf("%s", sTraceBuffer); Con::printf("%s", sTraceBuffer);
@ -477,12 +477,12 @@ void CodeInterpreter::parseArgs(U32 &ip)
} }
if (mExec.thisNamespace && mExec.thisNamespace->mName) if (mExec.thisNamespace && mExec.thisNamespace->mName)
{ {
dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), sizeof(sTraceBuffer) - dStrlen(sTraceBuffer), dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), (U32)(sizeof(sTraceBuffer) - dStrlen(sTraceBuffer)),
"%s::%s(", mExec.thisNamespace->mName, mThisFunctionName); "%s::%s(", mExec.thisNamespace->mName, mThisFunctionName);
} }
else else
{ {
dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), sizeof(sTraceBuffer) - dStrlen(sTraceBuffer), dSprintf(sTraceBuffer + dStrlen(sTraceBuffer), (U32)(sizeof(sTraceBuffer) - dStrlen(sTraceBuffer)),
"%s(", mThisFunctionName); "%s(", mThisFunctionName);
} }
for (S32 i = 0; i < wantedArgc; i++) for (S32 i = 0; i < wantedArgc; i++)
@ -2373,9 +2373,6 @@ OPCodeReturn CodeInterpreter::op_callfunc_pointer(U32 &ip)
{ {
const char* nsName = ""; const char* nsName = "";
Namespace::Entry::CallbackUnion * nsCb = &mNSEntry->cb;
const char * nsUsage = mNSEntry->mUsage;
#ifndef TORQUE_DEBUG #ifndef TORQUE_DEBUG
// [tom, 12/13/2006] This stops tools functions from working in the console, // [tom, 12/13/2006] This stops tools functions from working in the console,
// which is useful behavior when debugging so I'm ifdefing this out for debug builds. // which is useful behavior when debugging so I'm ifdefing this out for debug builds.
@ -2567,8 +2564,6 @@ OPCodeReturn CodeInterpreter::op_callfunc_this(U32 &ip)
} }
else else
{ {
Namespace::Entry::CallbackUnion * nsCb = &mNSEntry->cb;
const char * nsUsage = mNSEntry->mUsage;
const char* nsName = ns ? ns->mName : ""; const char* nsName = ns ? ns->mName : "";
#ifndef TORQUE_DEBUG #ifndef TORQUE_DEBUG
// [tom, 12/13/2006] This stops tools functions from working in the console, // [tom, 12/13/2006] This stops tools functions from working in the console,