Fixed issues causing newer compilers to complain.

This commit is contained in:
Cekis
2021-12-13 21:08:31 -05:00
parent 4c19ed30fe
commit 19ecf72b4c
4 changed files with 39 additions and 37 deletions

View File

@@ -35,7 +35,6 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)
set(CMAKE_CXX_EXTENSIONS OFF) set(CMAKE_CXX_EXTENSIONS OFF)
# shame on you cmake for auto-setting optimization flags! # shame on you cmake for auto-setting optimization flags!
set(CMAKE_CXX_FLAGS "")
set(CMAKE_CXX_FLAGS_DEBUG "") set(CMAKE_CXX_FLAGS_DEBUG "")
set(CMAKE_CXX_FLAGS_RELEASE "") set(CMAKE_CXX_FLAGS_RELEASE "")
set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "") set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "")
@@ -99,7 +98,7 @@ elseif (UNIX)
# RELWITHDEBINFO is used for building bins that produce profdata files # RELWITHDEBINFO is used for building bins that produce profdata files
# we only need the basics of our heavy optimizations here, i think - that and one of these flags # we only need the basics of our heavy optimizations here, i think - that and one of these flags
# breaks JNI when we profile # breaks JNI when we profile
set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "-Ofast -finline-hint-functions -fprofile-instr-generate") set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "${CMAKE_CXX_FLAGS} -Ofast -finline-hint-functions -fprofile-instr-generate")
# MINSIZEREL is used for profiled, flto builds # MINSIZEREL is used for profiled, flto builds
set(CMAKE_CXX_FLAGS_MINSIZEREL "${CMAKE_CXX_FLAGS_RELEASE} -flto -fwhole-program-vtables") set(CMAKE_CXX_FLAGS_MINSIZEREL "${CMAKE_CXX_FLAGS_RELEASE} -flto -fwhole-program-vtables")

View File

@@ -59,15 +59,15 @@ const int maxStringSize = 256;
const char version[] = "1.3 September 18, 2000"; const char version[] = "1.3 September 18, 2000";
// vars set by pragmas or via command line // vars set by pragmas or via command line
char drive[8]; // should be no more then 2 char "C:" // char drive[8]; // should be no more then 2 char "C:"
char directory[maxStringSize]; // char directory[maxStringSize];
char filename[maxStringSize]; // char filename[maxStringSize];
char extension[8]; // we'll truncate if the extension is more then 8 chars... // char extension[8]; // we'll truncate if the extension is more then 8 chars...
char inFileName[512]; char local_inFileName[512];
// switches to be sent to mIFF Compiler // switches to be sent to mIFF Compiler
char sourceBuffer[bufferSize]; char sourceBuffer[bufferSize];
char outFileName[512]; // x2 to combine filename, dir, and ext char local_outFileName[512]; // x2 to combine filename, dir, and ext
bool usePragma = false; bool usePragma = false;
bool useCCCP = false; bool useCCCP = false;
bool verboseMode = false; // default to non-verbose mode bool verboseMode = false; // default to non-verbose mode
@@ -92,8 +92,8 @@ enum errorType {
ERR_NONE = 0 ERR_NONE = 0
}; };
char err_msg[256]; char local_err_msg[256];
errorType errorFlag = ERR_NONE; // assume no error (default) errorType local_errorFlag = ERR_NONE; // assume no error (default)
// long and short name definitions for command line options // long and short name definitions for command line options
@@ -220,7 +220,7 @@ int main( int argc, // number of args in commandline
SetupSharedFoundation::remove(); SetupSharedFoundation::remove();
SetupSharedThread::remove(); SetupSharedThread::remove();
return static_cast<int> (errorFlag); return static_cast<int> (local_errorFlag);
} }
//--------------------------------------------------------------------------- //---------------------------------------------------------------------------
@@ -259,24 +259,24 @@ static void callbackFunction(void)
#endif #endif
errorFlag = evaluateArgs(); local_errorFlag = evaluateArgs();
if (ERR_NONE == errorFlag) if (ERR_NONE == local_errorFlag)
{ {
outfileHandler = new OutputFileHandler(outFileName); outfileHandler = new OutputFileHandler(local_outFileName);
MIFFCompile(sourceBuffer, inFileName); MIFFCompile(sourceBuffer, local_inFileName);
} }
else else
handleError(errorFlag); handleError(local_errorFlag);
if (outfileHandler) if (outfileHandler)
{ {
// only write output IF there was no error // only write output IF there was no error
if (ERR_NONE == errorFlag) if (ERR_NONE == local_errorFlag)
{ {
if (!outfileHandler->writeBuffer()) if (!outfileHandler->writeBuffer())
{ {
fprintf(stderr, "MIFF: failed to write output file \"%s\"\n", outFileName); fprintf(stderr, "MIFF: failed to write output file \"%s\"\n", local_outFileName);
errorFlag = ERR_WRITEERROR; local_errorFlag = ERR_WRITEERROR;
} }
} }
delete outfileHandler; delete outfileHandler;
@@ -323,13 +323,13 @@ static errorType evaluateArgs(void)
// setup input filename // setup input filename
strcpy(inFileName, CommandLine::getOptionString(SNAME_INPUT_FILE)); strcpy(local_inFileName, CommandLine::getOptionString(SNAME_INPUT_FILE));
// handle output filename spec // handle output filename spec
if (CommandLine::getOccurrenceCount(SNAME_OUTPUT_FILE)) if (CommandLine::getOccurrenceCount(SNAME_OUTPUT_FILE))
{ {
strcpy(outFileName, CommandLine::getOptionString(SNAME_OUTPUT_FILE)); strcpy(local_outFileName, CommandLine::getOptionString(SNAME_OUTPUT_FILE));
} }
else if (CommandLine::getOccurrenceCount(SNAME_PRAGMA_TARGET)) else if (CommandLine::getOccurrenceCount(SNAME_PRAGMA_TARGET))
{ {
@@ -342,15 +342,15 @@ static errorType evaluateArgs(void)
char *terminator; char *terminator;
// start with input file pathname // start with input file pathname
strcpy(outFileName, inFileName); strcpy(local_outFileName, local_inFileName);
// try to terminate at rightmost '.' // try to terminate at rightmost '.'
terminator = strrchr(outFileName, '.'); terminator = strrchr(local_outFileName, '.');
if (terminator) if (terminator)
*terminator = 0; *terminator = 0;
// append the default iff extension // append the default iff extension
strcat(outFileName, ".iff"); strcat(local_outFileName, ".iff");
} }
@@ -361,12 +361,12 @@ static errorType evaluateArgs(void)
// preprocess the input file // preprocess the input file
if (0 == preprocessSource(inFileName)) if (0 == preprocessSource(local_inFileName))
{ {
if (verboseMode) if (verboseMode)
{ {
sprintf(err_msg,"Now compiling %s...\n", inFileName); sprintf(local_err_msg,"Now compiling %s...\n", local_inFileName);
MIFFMessage(err_msg, 0); MIFFMessage(local_err_msg, 0);
} }
if (ERR_NONE == retVal) if (ERR_NONE == retVal)
retVal = loadInputToBuffer(sourceBuffer, bufferSize); retVal = loadInputToBuffer(sourceBuffer, bufferSize);
@@ -563,7 +563,7 @@ extern "C" void MIFFMessage(char *message, // nullptr terminated string to be
// Only call this via parser!!! // Only call this via parser!!!
extern "C" void MIFFSetError(void) extern "C" void MIFFSetError(void)
{ {
errorFlag = ERR_PARSER; local_errorFlag = ERR_PARSER;
} }
//--------------------------------------------------------------------------- //---------------------------------------------------------------------------
@@ -586,10 +586,10 @@ extern "C" int validateTargetFilename( char *targetFileName, // pointer to wher
unsigned maxTargetBufSize // size of the filename string buffer unsigned maxTargetBufSize // size of the filename string buffer
) )
{ {
if (strlen(outFileName) > maxTargetBufSize) if (strlen(local_outFileName) > maxTargetBufSize)
MIFFMessage("Internal error, increase string buffer size in parser.yac and recompile!", 1); MIFFMessage("Internal error, increase string buffer size in parser.yac and recompile!", 1);
strcpy(targetFileName, outFileName); strcpy(targetFileName, local_outFileName);
return(usePragma); return(usePragma);
} }
@@ -667,7 +667,7 @@ static int preprocessSource(char *sourceName)
extern "C" void MIFFSetIFFName(const char *newFileName) extern "C" void MIFFSetIFFName(const char *newFileName)
{ {
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return; return;
if (outfileHandler) if (outfileHandler)
@@ -676,7 +676,7 @@ extern "C" void MIFFSetIFFName(const char *newFileName)
extern "C" void MIFFinsertForm(const char *formName) extern "C" void MIFFinsertForm(const char *formName)
{ {
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return; return;
if (outfileHandler) if (outfileHandler)
@@ -685,7 +685,7 @@ extern "C" void MIFFinsertForm(const char *formName)
extern "C" void MIFFinsertChunk(const char *chunkName) extern "C" void MIFFinsertChunk(const char *chunkName)
{ {
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return; return;
if (outfileHandler) if (outfileHandler)
@@ -694,7 +694,7 @@ extern "C" void MIFFinsertChunk(const char *chunkName)
extern "C" void MIFFinsertChunkData(void * buffer, unsigned bufferSize) extern "C" void MIFFinsertChunkData(void * buffer, unsigned bufferSize)
{ {
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return; return;
if (outfileHandler) if (outfileHandler)
@@ -705,7 +705,7 @@ extern "C" int MIFFloadRawData(char *fname, void * buffer, unsigned maxBufferSiz
{ {
int sizeRead = -1; int sizeRead = -1;
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return(sizeRead); // should be -1 return(sizeRead); // should be -1
InputFileHandler * inFileName = new InputFileHandler(fname); InputFileHandler * inFileName = new InputFileHandler(fname);
@@ -725,7 +725,7 @@ extern "C" int MIFFloadRawData(char *fname, void * buffer, unsigned maxBufferSiz
extern "C" void MIFFexitChunk(void) extern "C" void MIFFexitChunk(void)
{ {
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return; return;
if (outfileHandler) if (outfileHandler)
@@ -733,7 +733,7 @@ extern "C" void MIFFexitChunk(void)
} }
extern "C" void MIFFexitForm(void) extern "C" void MIFFexitForm(void)
{ {
if (ERR_NONE != errorFlag) if (ERR_NONE != local_errorFlag)
return; return;
if (outfileHandler) if (outfileHandler)

View File

@@ -8,6 +8,8 @@
#ifndef REMOTE_DEBUG_H #ifndef REMOTE_DEBUG_H
#define REMOTE_DEBUG_H #define REMOTE_DEBUG_H
#include <string>
// ====================================================================== // ======================================================================
/** This class is both a platform and network-protocol independent way of /** This class is both a platform and network-protocol independent way of

View File

@@ -15,6 +15,7 @@
#include <set> #include <set>
#include <queue> #include <queue>
#include <list> #include <list>
#include <string>
namespace Unicode namespace Unicode
{ {