Rampoina
/
MegaGlest
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

code cleanup from cppcheck

This commit is contained in:
Mark Vejvoda 2013-06-15 01:43:44 +00:00
parent c646d0371c
commit 1b3ee1503b
9 changed files with 19 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
source/g3d_viewer/main.cpp
View File

@ -423,7 +423,7 @@ MainWindow::MainWindow( std::pair<string,vector<string> > unitToLoad,
if(modelPath != "") {
fileDialog->SetPath(ToUnicode(modelPath));
}
string userData = Config::getInstance().getString("UserData_Root","");
string userData = config.getString("UserData_Root","");
if(userData != "") {
endPathWithSlash(userData);
}

2
source/glest_game/facilities/components.cpp
View File

@ -379,7 +379,7 @@ void GraphicListBox::init(int x, int y, int w, int h, Vec3f textColor){
}
const string & GraphicListBox::getTextNativeTranslation() {
if(this->translated_items.size() == 0 ||
if(this->translated_items.empty() == true ||
this->selectedItemIndex < 0 ||
this->selectedItemIndex >= this->translated_items.size() ||
this->items.size() != this->translated_items.size()) {

8
source/glest_game/main/main.cpp
View File

@ -2542,8 +2542,6 @@ void runTechTranslationExtractionForPath(string techPath, string techName,
}
if(factions.empty() == false) {
bool techtree_errors = false;
std::map<string,vector<pair<string, string> > > loadedFileList;
//vector<string> pathList = config.getPathListForType(ptTechs,"");
vector<string> pathList;
@ -2652,12 +2650,6 @@ void runTechTranslationExtraction(int argc, char** argv) {
//disableBacktrace=true;
printf("====== Started Translation Extraction ======\n");
bool purgeDuplicateFiles = false;
bool showDuplicateFiles = true;
bool purgeUnusedFiles = false;
bool svnPurgeFiles = false;
double purgedMegaBytes=0;
Config &config = Config::getInstance();
// Did the user pass a specific list of factions to validate?

6
source/glest_game/menu/menu_state_about.cpp
View File

@ -40,6 +40,12 @@ MenuStateAbout::MenuStateAbout(Program *program, MainMenu *mainMenu) :
adjustModelText = true;
customModTextureX = 0;
customModTextureY = 0;
customModTextureW = 0;
customModTextureH = 0;
customModTextureAlpha = 0.f;
string additionalCredits= loadAdditionalCredits();
//init

2
source/glest_game/menu/menu_state_custom_game.cpp
View File

@ -3191,7 +3191,7 @@ void MenuStateCustomGame::loadGameSettings(GameSettings *gameSettings,bool force
Lang &lang= Lang::getInstance();
gameSettings->setNetworkPlayerLanguages(slotIndex, lang.getLanguage());
}
else if(serverInterface->getSlot(i) != NULL) {
else if(serverInterface != NULL && serverInterface->getSlot(i) != NULL) {
gameSettings->setNetworkPlayerLanguages(slotIndex, serverInterface->getSlot(i)->getNetworkPlayerLanguage());
}

2
source/glest_game/network/network_interface.cpp
View File

@ -56,6 +56,8 @@ void NetworkInterface::init() {
networkGameDataSynchCheckOkTile=false;
networkGameDataSynchCheckOkTech=false;
receivedDataSynchCheck=false;
gameSettings = GameSettings();
}
NetworkInterface::~NetworkInterface() {

8
source/glest_game/type_instances/faction.cpp
View File

@ -275,11 +275,10 @@ bool FactionThread::isSignalPathfinderCompleted(int frameIndex) {
if(getRunningStatus() == false) {
return true;
}
bool result = false;
static string mutexOwnerId = string(__FILE__) + string("_") + intToStr(__LINE__);
MutexSafeWrapper safeMutex(triggerIdMutex,mutexOwnerId);
//bool result = (event != NULL ? event->eventCompleted : true);
result = (this->frameIndex.first == frameIndex && this->frameIndex.second == true);
bool result = (this->frameIndex.first == frameIndex && this->frameIndex.second == true);
//if(SystemFlags::VERBOSE_MODE_ENABLED) printf("In [%s::%s Line: %d] worker thread this = %p, this->frameIndex.first = %d, this->frameIndex.second = %d\n",__FILE__,__FUNCTION__,__LINE__,this,this->frameIndex.first,this->frameIndex.second);
@ -316,11 +315,10 @@ void FactionThread::execute() {
break;
}
int currentTriggeredFrameIndex = 0;
static string mutexOwnerId = string(__FILE__) + string("_") + intToStr(__LINE__);
MutexSafeWrapper safeMutex(triggerIdMutex,mutexOwnerId);
bool executeTask = (this->frameIndex.first >= 0);
currentTriggeredFrameIndex = this->frameIndex.first;
int currentTriggeredFrameIndex = this->frameIndex.first;
//if(SystemFlags::VERBOSE_MODE_ENABLED) printf("In [%s::%s Line: %d] frameIndex = %d this = %p executeTask = %d\n",__FILE__,__FUNCTION__,__LINE__,frameIndex.first, this, executeTask);
@ -482,6 +480,8 @@ void Faction::init() {
control = ctClosed;
overridePersonalityType = fpt_EndCount;
upgradeManager = UpgradeManager();
}
Faction::~Faction() {

5
source/glest_game/type_instances/unit.cpp
View File

@ -2219,7 +2219,7 @@ void Unit::updateAttackBoostProgress(const Game* game) {
for(unsigned int i = 0; i < currentAttackBoostOriginatorEffect.currentAttackBoostUnits.size(); ++i) {
int unitnum = currentAttackBoostOriginatorEffect.currentAttackBoostUnits[i];
printf("affected unit #: %d - %d\n",i,unitnum);
printf("affected unit #: %u - %d\n",i,unitnum);
}
}
}
@ -3850,10 +3850,9 @@ void Unit::logSynchDataCommon(string file,int line,string source,bool threadedMo
lastFile = file;
lastSource = source;
string logDataText = "";
char szBufDataText[8096]="";
snprintf(szBufDataText,8096,"----------------------------------- START [FRAME %d UNIT: %d - %s] ------------------------------------------------\n",getFrameCount(),this->id,this->getType()->getName(false).c_str());
logDataText = szBufDataText;
string logDataText = szBufDataText;
snprintf(szBufDataText,8096,"[%s::%d]\n",extractFileFromDirectoryPath(file).c_str(),line);
logDataText += szBufDataText;

4
source/shared_lib/sources/platform/common/platform_common.cpp
View File

@ -551,7 +551,7 @@ void updatePathClimbingParts(string &path) {
if(pos != string::npos && pos != 0) {
string orig = path;
path.erase(pos,2);
pos--;
//pos--;
pos = path.find("/./");
if(pos != string::npos && pos != 0) {
@ -564,7 +564,7 @@ void updatePathClimbingParts(string &path) {
if(pos != string::npos && pos != 0) {
string orig = path;
path.erase(pos,2);
pos--;
//pos--;
pos = path.find("\\.\\");
if(pos != string::npos && pos != 0) {