diff options
author | Andrei Karas <akaras@inbox.ru> | 2018-11-26 00:51:57 +0300 |
---|---|---|
committer | Andrei Karas <akaras@inbox.ru> | 2018-11-27 20:33:15 +0300 |
commit | 0c20db8d108c4123fcc3ecb343759e68567a4d4e (patch) | |
tree | f577edeae7b4866d68c084a1eaa229e98c80ea8f | |
parent | 2be4fc5b4eee97938657c9c8be73c4b91c8b9826 (diff) | |
download | plus-0c20db8d108c4123fcc3ecb343759e68567a4d4e.tar.gz plus-0c20db8d108c4123fcc3ecb343759e68567a4d4e.tar.bz2 plus-0c20db8d108c4123fcc3ecb343759e68567a4d4e.tar.xz plus-0c20db8d108c4123fcc3ecb343759e68567a4d4e.zip |
Remove extra ; from different code.
233 files changed, 1085 insertions, 1083 deletions
diff --git a/src/actions/actiondef.h b/src/actions/actiondef.h index 636d6eb6c..8d2cdd1e1 100644 --- a/src/actions/actiondef.h +++ b/src/actions/actiondef.h @@ -39,7 +39,7 @@ INPUT_INPUTMANAGER_H #define callYellowBar(name) \ GameModifiers::name(!inputManager.isActionActive( \ InputAction::STOP_ATTACK)); \ - return true; + return true #define callYellowBarCond(name) \ if (!settings.disableGameModifiers) \ @@ -48,6 +48,6 @@ INPUT_INPUTMANAGER_H InputAction::STOP_ATTACK)); \ return true; \ } \ - return false; + return false #endif // ACTIONS_ACTIONDEF_H diff --git a/src/actormanager.cpp b/src/actormanager.cpp index 974eef64a..59bfd5bb3 100644 --- a/src/actormanager.cpp +++ b/src/actormanager.cpp @@ -332,7 +332,7 @@ Being *ActorManager::createBeing(const BeingId id, case ActorType::FloorItem: case ActorType::Avatar: case ActorType::Unknown: - reportAlways("CreateBeing for unknown type %d", CAST_S32(type)); + reportAlways("CreateBeing for unknown type %d", CAST_S32(type)) break; } @@ -382,7 +382,7 @@ FloorItem *ActorManager::createItem(const BeingId id, void ActorManager::destroy(ActorSprite *const actor) { - returnNullptrV(actor); + returnNullptrV(actor) if (actor == localPlayer) return; @@ -392,7 +392,7 @@ void ActorManager::destroy(ActorSprite *const actor) void ActorManager::erase(ActorSprite *const actor) { - returnNullptrV(actor); + returnNullptrV(actor) if (actor == localPlayer) return; @@ -405,7 +405,7 @@ void ActorManager::erase(ActorSprite *const actor) void ActorManager::undelete(const ActorSprite *const actor) { - returnNullptrV(actor); + returnNullptrV(actor) if (actor == localPlayer) return; @@ -682,7 +682,7 @@ FloorItem *ActorManager::findItem(const BeingId id) const if (it != mActorsIdMap.end()) { ActorSprite *const actor = (*it).second; - returnNullptr(nullptr, actor); + returnNullptr(nullptr, actor) if (actor->getId() == id && actor->getType() == ActorType::FloorItem) { @@ -1922,14 +1922,14 @@ void ActorManager::removePickupItem(const std::string &name) void ActorManager::addAttackMob(const std::string &name) { - addMobToList(name, AttackMob); + addMobToList(name, AttackMob) rebuildPriorityAttackMobs(); storeAttackList(); } void ActorManager::addPriorityAttackMob(const std::string &name) { - addMobToList(name, PriorityAttackMob); + addMobToList(name, PriorityAttackMob) storeAttackList(); } @@ -1944,7 +1944,7 @@ void ActorManager::addIgnoreAttackMob(const std::string &name) void ActorManager::addPickupItem(const std::string &name) { - addMobToList(name, PickupItem); + addMobToList(name, PickupItem) rebuildPickupItems(); storeAttackList(); } @@ -1959,17 +1959,17 @@ void ActorManager::addIgnorePickupItem(const std::string &name) void ActorManager::rebuildPriorityAttackMobs() { - rebuildMobsList(PriorityAttackMob); + rebuildMobsList(PriorityAttackMob) } void ActorManager::rebuildAttackMobs() { - rebuildMobsList(AttackMob); + rebuildMobsList(AttackMob) } void ActorManager::rebuildPickupItems() { - rebuildMobsList(PickupItem); + rebuildMobsList(PickupItem) } int ActorManager::getIndexByName(const std::string &name, @@ -2019,9 +2019,9 @@ void ActorManager::loadAttackList() std::list<std::string>::const_iterator i; std::list<std::string>::const_iterator i_end; - loadList("attackPriorityMobs", PriorityAttackMob); - loadList("attackMobs", AttackMob); - loadList("ignoreAttackMobs", IgnoreAttackMob); + loadList("attackPriorityMobs", PriorityAttackMob) + loadList("attackMobs", AttackMob) + loadList("ignoreAttackMobs", IgnoreAttackMob) if (!empty) { mAttackMobs.push_back(""); @@ -2029,8 +2029,8 @@ void ActorManager::loadAttackList() } empty = false; - loadList("pickupItems", PickupItem); - loadList("ignorePickupItems", IgnorePickupItem); + loadList("pickupItems", PickupItem) + loadList("ignorePickupItems", IgnorePickupItem) if (!empty) { mPickupItems.push_back(""); @@ -2101,7 +2101,7 @@ Being *ActorManager::cloneBeing(const Being *const srcBeing, const int dx, const int dy, const int id) { - returnNullptr(nullptr, srcBeing); + returnNullptr(nullptr, srcBeing) Being *const dstBeing = actorManager->createBeing(fromInt( toInt(srcBeing->getId(), int) + id, BeingId), ActorType::Player, diff --git a/src/being/being.cpp b/src/being/being.cpp index d43912258..7fc270b22 100644 --- a/src/being/being.cpp +++ b/src/being/being.cpp @@ -346,16 +346,16 @@ Being::~Being() mSpriteDraw = nullptr; for_each_badges() - delete2(mBadges[f]); - - delete2(mSpeechBubble); - delete2(mDispName); - delete2(mText); - delete2(mEmotionSprite); - delete2(mAnimationEffect); - delete2(mCastingEffect); + delete2(mBadges[f]) + + delete2(mSpeechBubble) + delete2(mDispName) + delete2(mText) + delete2(mEmotionSprite) + delete2(mAnimationEffect) + delete2(mCastingEffect) mBadgesCount = 0; - delete2(mChat); + delete2(mChat) removeHorse(); removeAllItemsParticles(); @@ -509,7 +509,7 @@ void Being::setSubtype(const BeingTypeId subtype, case ActorType::FloorItem: default: reportAlways("Wrong being type %d in setSubType", - CAST_S32(mType)); + CAST_S32(mType)) break; } } @@ -1117,7 +1117,7 @@ void Being::handleSkill(Being *restrict const victim, void Being::showNameBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Name]); + delete2(mBadges[BadgeIndex::Name]) if (show && !mName.empty() && mShowBadges != BadgeDrawType::Hide) @@ -1177,7 +1177,7 @@ void Being::setShowName(const bool doShowName) restrict2 void Being::showGuildBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Guild]); + delete2(mBadges[BadgeIndex::Guild]) if (show && !mGuildName.empty() && mShowBadges != BadgeDrawType::Hide) @@ -1204,7 +1204,7 @@ void Being::setGuildName(const std::string &restrict name) restrict2 void Being::showClanBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Clan]); + delete2(mBadges[BadgeIndex::Clan]) if (show && !mClanName.empty() && mShowBadges != BadgeDrawType::Hide) @@ -1860,7 +1860,7 @@ void Being::logic() restrict2 if (A_UNLIKELY(mCastEndTime != 0 && mCastEndTime < tick_time)) { mCastEndTime = 0; - delete2(mCastingEffect); + delete2(mCastingEffect) } if (A_UNLIKELY(mAnimationEffect)) @@ -2580,7 +2580,7 @@ void Being::showName() restrict2 if (mName.empty()) return; - delete2(mDispName); + delete2(mDispName) if (mHideErased && playerRelations.getRelation(mName) == Relation::ERASED) return; @@ -3645,7 +3645,7 @@ void Being::setGender(const GenderT gender) restrict2 void Being::showGmBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Gm]); + delete2(mBadges[BadgeIndex::Gm]) if (show && mIsGM && mShowBadges != BadgeDrawType::Hide && @@ -4945,7 +4945,7 @@ void Being::removeSpecialEffect() restrict2 mChildParticleEffects.removeLocally(mSpecialParticle); mSpecialParticle = nullptr; } - delete2(mAnimationEffect); + delete2(mAnimationEffect) } void Being::updateAwayEffect() restrict2 @@ -5023,7 +5023,7 @@ void Being::setTileCoords(const int x, const int y) restrict2 void Being::setMap(Map *restrict const map) restrict2 { mCastEndTime = 0; - delete2(mCastingEffect); + delete2(mCastingEffect) ActorSprite::setMap(map); if (mMap != nullptr) { @@ -5196,7 +5196,7 @@ void Being::setTeamId(const uint16_t teamId) restrict2 void Being::showTeamBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Team]); + delete2(mBadges[BadgeIndex::Team]) if (show && mTeamId != 0U && mShowBadges != BadgeDrawType::Hide) @@ -5225,7 +5225,7 @@ void Being::showBadges(const bool show) restrict2 void Being::showPartyBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Party]); + delete2(mBadges[BadgeIndex::Party]) if (show && !mPartyName.empty() && mShowBadges != BadgeDrawType::Hide) @@ -5254,7 +5254,7 @@ void Being::setPartyName(const std::string &restrict name) restrict2 void Being::showShopBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Shop]); + delete2(mBadges[BadgeIndex::Shop]) if (show && mShop && mShowBadges != BadgeDrawType::Hide) @@ -5273,7 +5273,7 @@ void Being::showShopBadge(const bool show) restrict2 void Being::showInactiveBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Inactive]); + delete2(mBadges[BadgeIndex::Inactive]) if (show && mInactive && mShowBadges != BadgeDrawType::Hide) @@ -5292,7 +5292,7 @@ void Being::showInactiveBadge(const bool show) restrict2 void Being::showAwayBadge(const bool show) restrict2 { - delete2(mBadges[BadgeIndex::Away]); + delete2(mBadges[BadgeIndex::Away]) if (show && mAway && mShowBadges != BadgeDrawType::Hide) @@ -5382,7 +5382,7 @@ void Being::addCast(const int dstX, SkillData *const data = skillDialog->getSkillDataByLevel( skillId, skillLevel); - delete2(mCastingEffect); + delete2(mCastingEffect) if (data != nullptr) { const std::string castingAnimation = data->castingAnimation; @@ -5397,7 +5397,7 @@ void Being::addCast(const int dstX, else { reportAlways("Want to draw casting for unknown skill %d", - skillId); + skillId) } } @@ -5556,7 +5556,7 @@ void Being::setLanguageId(const int lang) restrict2 noexcept2 { if (lang != mLanguageId) { - delete2(mBadges[BadgeIndex::Lang]); + delete2(mBadges[BadgeIndex::Lang]) const std::string &badge = LanguageDb::getIcon(lang); if (!badge.empty()) { diff --git a/src/being/castingeffect.cpp b/src/being/castingeffect.cpp index aa3e6a14d..17b0b177f 100644 --- a/src/being/castingeffect.cpp +++ b/src/being/castingeffect.cpp @@ -60,13 +60,13 @@ CastingEffect::CastingEffect(const int skillId, reportAlways("Skill %d/%d casting animation '%s' load failed", skillId, skillLevel, - animation.c_str()); + animation.c_str()) } } CastingEffect::~CastingEffect() { - delete2(mSprite); + delete2(mSprite) } void CastingEffect::draw(Graphics *const graphics, diff --git a/src/being/compoundsprite.cpp b/src/being/compoundsprite.cpp index 843c3b6b6..ebb4466b3 100644 --- a/src/being/compoundsprite.cpp +++ b/src/being/compoundsprite.cpp @@ -290,7 +290,7 @@ void CompoundSprite::removeSprite(const int layer) if (mSprites[layer] == nullptr) return; - delete2(mSprites[layer]); + delete2(mSprites[layer]) mNeedsRedraw = true; } @@ -305,7 +305,7 @@ void CompoundSprite::clear() mNeedsRedraw = true; delete_all(imagesCache); imagesCache.clear(); - delete2(mCacheItem); + delete2(mCacheItem) mLastTime = 0; } @@ -368,7 +368,7 @@ void CompoundSprite::redraw() const drawSpritesSDL(graphics, posX, posY); - delete2(graphics); + delete2(graphics) SDL_SetAlpha(surface, 0, SDL_ALPHA_OPAQUE); diff --git a/src/being/localplayer.cpp b/src/being/localplayer.cpp index d344926fa..ce0ab1e27 100644 --- a/src/being/localplayer.cpp +++ b/src/being/localplayer.cpp @@ -238,7 +238,7 @@ LocalPlayer::~LocalPlayer() soundManager.volumeRestore(); delete2(mAwayDialog) } - delete2(mAwayListener); + delete2(mAwayListener) } void LocalPlayer::logic() @@ -404,7 +404,7 @@ void LocalPlayer::setAction(const BeingActionT &action, { // TRANSLATORS: chat message after death debugMsg(strprintf(_("You were killed by %s."), - mLastHitFrom.c_str())); + mLastHitFrom.c_str())) mLastHitFrom.clear(); } setTarget(nullptr); @@ -2340,12 +2340,12 @@ void LocalPlayer::setFollow(const std::string &player) { // TRANSLATORS: follow command message std::string msg = strprintf(_("Follow: %s"), player.c_str()); - debugMsg(msg); + debugMsg(msg) } else { // TRANSLATORS: follow command message - debugMsg(_("Follow canceled")); + debugMsg(_("Follow canceled")) } } @@ -2356,12 +2356,12 @@ void LocalPlayer::setImitate(const std::string &player) { // TRANSLATORS: imitate command message std::string msg = strprintf(_("Imitation: %s"), player.c_str()); - debugMsg(msg); + debugMsg(msg) } else { // TRANSLATORS: imitate command message - debugMsg(_("Imitation canceled")); + debugMsg(_("Imitation canceled")) } } @@ -2370,12 +2370,12 @@ void LocalPlayer::cancelFollow() if (!mPlayerFollowed.empty()) { // TRANSLATORS: cancel follow message - debugMsg(_("Follow canceled")); + debugMsg(_("Follow canceled")) } if (!mPlayerImitated.empty()) { // TRANSLATORS: cancel follow message - debugMsg(_("Imitation canceled")); + debugMsg(_("Imitation canceled")) } mPlayerFollowed.clear(); mPlayerImitated.clear(); @@ -2733,7 +2733,7 @@ void LocalPlayer::checkNewName(Being *const being) if (!mWaitFor.empty() && mWaitFor == nick) { // TRANSLATORS: wait player/monster message - debugMsg(strprintf(_("You see %s"), mWaitFor.c_str())); + debugMsg(strprintf(_("You see %s"), mWaitFor.c_str())) soundManager.playGuiSound(SOUND_INFO); mWaitFor.clear(); } diff --git a/src/being/playerinfo.cpp b/src/being/playerinfo.cpp index 32a39e0c7..cb8fc195a 100644 --- a/src/being/playerinfo.cpp +++ b/src/being/playerinfo.cpp @@ -436,7 +436,7 @@ void init() void deinit() { clearInventory(); - delete2(mMercenary); + delete2(mMercenary) mPetBeingId = BeingId_zero; } @@ -460,9 +460,9 @@ bool isTalking() void gameDestroyed() { - delete2(mInventory); - delete2(mEquipment); - delete2(mCartInventory); + delete2(mInventory) + delete2(mEquipment) + delete2(mCartInventory) } void stateChange(const StateT state) diff --git a/src/configmanager.cpp b/src/configmanager.cpp index ad3a5470a..6278ed3de 100644 --- a/src/configmanager.cpp +++ b/src/configmanager.cpp @@ -87,7 +87,7 @@ void ConfigManager::initServerConfig(const std::string &serverName) else { reportAlways("Error creating server config: %s", - configPath.c_str()); + configPath.c_str()) } const bool val = Client::isTmw(); @@ -160,7 +160,7 @@ void ConfigManager::initConfiguration() if (configFile == nullptr) { reportAlways("Can't create %s. Using defaults.", - configPath.c_str()); + configPath.c_str()) } else { @@ -429,10 +429,10 @@ void ConfigManager::checkConfigVersion() const std::string prefix = std::string(); #endif // USE_SDL2 - unassignKey("keyDirectUp", "k108"); - unassignKey("keyDirectDown", "k59"); - unassignKey("keyDirectLeft", "k107"); - unassignKey("keyDirectRight", "k39"); + unassignKey("keyDirectUp", "k108") + unassignKey("keyDirectDown", "k59") + unassignKey("keyDirectLeft", "k107") + unassignKey("keyDirectRight", "k39") } if (version < 12) { @@ -443,7 +443,7 @@ void ConfigManager::checkConfigVersion() const std::string prefix = std::string(); #endif // USE_SDL2 - unassignKey("keyAttack", "k120"); + unassignKey("keyAttack", "k120") } if (version < 13) diff --git a/src/configuration.cpp b/src/configuration.cpp index 85c69e28b..79bb5041e 100644 --- a/src/configuration.cpp +++ b/src/configuration.cpp @@ -443,7 +443,7 @@ int Configuration::getIntValue(const std::string &key) const reportAlways( "%s: No integer value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } } else @@ -462,7 +462,7 @@ int Configuration::resetIntValue(const std::string &key) { reportAlways("%s: No integer value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } else { @@ -477,7 +477,7 @@ int Configuration::resetIntValue(const std::string &key) { reportAlways("%s: No integer value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } } setValue(key, defaultValue); @@ -527,7 +527,7 @@ std::string Configuration::getStringValue(const std::string &key) const { reportAlways("%s: No string value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } } else @@ -583,7 +583,7 @@ float Configuration::getFloatValue(const std::string &key) const { reportAlways("%s: No float value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } } else @@ -651,7 +651,7 @@ bool Configuration::getBoolValue(const std::string &key) const reportAlways( "%s: No boolean value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } } else @@ -672,7 +672,7 @@ bool Configuration::resetBoolValue(const std::string &key) { reportAlways("%s: No boolean value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } else { @@ -686,7 +686,7 @@ bool Configuration::resetBoolValue(const std::string &key) { reportAlways("%s: No boolean value in registry for key %s", mConfigPath.c_str(), - key.c_str()); + key.c_str()) } } @@ -870,7 +870,7 @@ void Configuration::write() if (testFile == nullptr) { reportAlways("Configuration::write() couldn't open %s for writing", - mConfigPath.c_str()); + mConfigPath.c_str()) BLOCK_END("Configuration::write") return; } diff --git a/src/defaults.cpp b/src/defaults.cpp index 27954fa9d..33ca98b76 100644 --- a/src/defaults.cpp +++ b/src/defaults.cpp @@ -84,7 +84,7 @@ VariableData* createData(const bool defData) #define AddDEF(key, value) \ configData.insert(std::pair<std::string, VariableData*> \ - (key, createData(value))); + (key, createData(value))) void setConfigDefaults(Configuration &cfg) { diff --git a/src/effectmanager.cpp b/src/effectmanager.cpp index e995280ff..b13699854 100644 --- a/src/effectmanager.cpp +++ b/src/effectmanager.cpp @@ -44,7 +44,7 @@ EffectManager::EffectManager() : logger->log1("Effects are now loading"); loadXmlFile(paths.getStringValue("effectsFile"), SkipError_false); loadXmlFile(paths.getStringValue("effectsPatchFile"), SkipError_true); - loadXmlDir("effectsPatchDir", loadXmlFile); + loadXmlDir("effectsPatchDir", loadXmlFile) } void EffectManager::loadXmlFile(const std::string &fileName, @@ -143,7 +143,7 @@ bool EffectManager::trigger(const int id, return rValue; } } - reportAlways("Missing effect %d", id); + reportAlways("Missing effect %d", id) BLOCK_END("EffectManager::trigger") return rValue; } @@ -174,7 +174,7 @@ Particle *EffectManager::triggerReturn(const int id, return rValue; } } - reportAlways("Missing effect %d", id); + reportAlways("Missing effect %d", id) return rValue; } @@ -207,7 +207,7 @@ bool EffectManager::trigger(const int id, return rValue; } } - reportAlways("Missing effect %d", id); + reportAlways("Missing effect %d", id) return rValue; } diff --git a/src/eventsmanager.cpp b/src/eventsmanager.cpp index 090cfdb82..986d9cf9e 100644 --- a/src/eventsmanager.cpp +++ b/src/eventsmanager.cpp @@ -537,7 +537,7 @@ void EventsManager::logEvent(const SDL_Event &event) logger->assertLog("event: other: %u", CAST_U32(event.type)); break; - }; + } } #ifdef USE_SDL2 diff --git a/src/fs/files.cpp b/src/fs/files.cpp index ce7f663a3..ccec9ab87 100644 --- a/src/fs/files.cpp +++ b/src/fs/files.cpp @@ -236,7 +236,7 @@ bool Files::loadTextFileLocal(const std::string &fileName, if (!file.is_open()) { reportAlways("Couldn't load text file: %s", - fileName.c_str()); + fileName.c_str()) return false; } @@ -262,7 +262,7 @@ void Files::saveTextFile(const std::string &path, else { reportAlways("Error opening file for writing: %s", - fileName.c_str()); + fileName.c_str()) } file.close(); } diff --git a/src/fs/paths.cpp b/src/fs/paths.cpp index 8fb57330d..6c8d9f104 100644 --- a/src/fs/paths.cpp +++ b/src/fs/paths.cpp @@ -139,7 +139,7 @@ void prepareFsPath(std::string &path) { reportAlways("Path can be improved: '%s' -> '%s'", path2.c_str(), - path.c_str()); + path.c_str()) } #endif } diff --git a/src/fs/virtfs/fs.cpp b/src/fs/virtfs/fs.cpp index c4faf02ed..697f28579 100644 --- a/src/fs/virtfs/fs.cpp +++ b/src/fs/virtfs/fs.cpp @@ -42,9 +42,9 @@ const char *dirSeparator = nullptr; #ifdef UNITTESTS -#define reportNonTests logger->log +#define reportNonTests(a, b) logger->log(a, b); #else // UNITTESTS -#define reportNonTests reportAlways +#define reportNonTests(a, b) reportAlways(a, b) #endif // UNITTESTS namespace VirtFs @@ -126,7 +126,7 @@ namespace VirtFs if (checkPath(name) == false) { reportAlways("FsDir::exists invalid path: %s", - name.c_str()); + name.c_str()) return false; } @@ -150,7 +150,7 @@ namespace VirtFs if (checkPath(dirName) == false) { reportAlways("VirtFs::enumerateFiles invalid path: %s", - dirName.c_str()); + dirName.c_str()) return list; } @@ -175,7 +175,7 @@ namespace VirtFs if (checkPath(dirName) == false) { reportAlways("VirtFs::enumerateFiles invalid path: %s", - dirName.c_str()); + dirName.c_str()) return; } @@ -197,7 +197,7 @@ namespace VirtFs if (checkPath(dirName) == false) { reportAlways("VirtFs::enumerateFiles invalid path: %s", - dirName.c_str()); + dirName.c_str()) return; } @@ -220,7 +220,7 @@ namespace VirtFs if (checkPath(dirName) == false) { reportAlways("VirtFs::enumerateFiles invalid path: %s", - dirName.c_str()); + dirName.c_str()) return; } @@ -241,7 +241,7 @@ namespace VirtFs if (checkPath(name) == false) { reportAlways("VirtFs::isDirectory invalid path: %s", - name.c_str()); + name.c_str()) return false; } std::string dirName = STD_MOVE(name); @@ -276,7 +276,7 @@ namespace VirtFs if (checkPath(filename) == false) { reportAlways("VirtFs::openRead invalid path: %s", - filename.c_str()); + filename.c_str()) return nullptr; } FOR_EACH (STD_VECTOR<FsEntry*>::const_iterator, it, mEntries) @@ -295,7 +295,7 @@ namespace VirtFs if (checkPath(filename) == false) { reportAlways("VirtFs::openWrite invalid path: %s", - filename.c_str()); + filename.c_str()) return nullptr; } FOR_EACH (STD_VECTOR<FsEntry*>::const_iterator, it, mEntries) @@ -314,7 +314,7 @@ namespace VirtFs if (checkPath(filename) == false) { reportAlways("VirtFs::openAppend invalid path: %s", - filename.c_str()); + filename.c_str()) return nullptr; } FOR_EACH (STD_VECTOR<FsEntry*>::const_iterator, it, mEntries) @@ -347,7 +347,7 @@ namespace VirtFs { if (newDir.find(".zip") != std::string::npos) { - reportAlways("Called FsDir::mount with zip archive"); + reportAlways("Called FsDir::mount with zip archive") return false; } std::string rootDir = newDir; @@ -366,7 +366,7 @@ namespace VirtFs if (entry != nullptr) { reportAlways("VirtFs::mount already exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } if (subDir.empty()) @@ -396,7 +396,7 @@ namespace VirtFs if (Files::existsLocal(newDir) == false) { reportNonTests("VirtFs::mount directory not exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } return mountDirInternal(newDir, dirSeparator, append); @@ -411,7 +411,7 @@ namespace VirtFs if (Files::existsLocal(newDir) == false) { reportNonTests("VirtFs::mount directory not exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } return mountDirInternal(newDir, subDir, append); @@ -519,13 +519,13 @@ namespace VirtFs prepareFsPath(oldDir); if (oldDir.find(".zip") != std::string::npos) { - reportAlways("Called unmount with zip archive"); + reportAlways("Called unmount with zip archive") return false; } if (unmountDirInternal(oldDir, std::string()) == false) { reportAlways("VirtFs::unmountDir not exists: %s", - oldDir.c_str()); + oldDir.c_str()) return false; } return true; @@ -537,14 +537,14 @@ namespace VirtFs prepareFsPath(oldDir); if (oldDir.find(".zip") != std::string::npos) { - reportAlways("Called unmount with zip archive"); + reportAlways("Called unmount with zip archive") return false; } prepareFsPath(subDir); if (unmountDirInternal(oldDir, subDir) == false) { reportAlways("VirtFs::unmountDir not exists: %s", - oldDir.c_str()); + oldDir.c_str()) return false; } return true; @@ -555,7 +555,7 @@ namespace VirtFs prepareFsPath(oldDir); if (oldDir.find(".zip") != std::string::npos) { - reportAlways("Called unmount with zip archive"); + reportAlways("Called unmount with zip archive") return false; } if (unmountDirInternal(oldDir, std::string()) == false) @@ -573,7 +573,7 @@ namespace VirtFs prepareFsPath(oldDir); if (oldDir.find(".zip") != std::string::npos) { - reportAlways("Called unmount with zip archive"); + reportAlways("Called unmount with zip archive") return false; } prepareFsPath(subDir); @@ -593,19 +593,19 @@ namespace VirtFs if (Files::existsLocal(newDir) == false) { reportNonTests("FsZip::mount file not exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } if (findLast(newDir, ".zip") == false) { reportAlways("Called VirtFs::mount without " - "zip archive"); + "zip archive") return false; } if (searchByRootInternal(newDir, std::string()) != nullptr) { reportAlways("FsZip::mount already exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } ZipEntry *const entry = new ZipEntry(newDir, @@ -630,13 +630,13 @@ namespace VirtFs if (Files::existsLocal(newDir) == false) { reportNonTests("FsZip::mount file not exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } if (findLast(newDir, ".zip") == false) { reportAlways("Called VirtFs::mount without " - "zip archive"); + "zip archive") return false; } prepareFsPath(subDir); @@ -652,7 +652,7 @@ namespace VirtFs if (searchByRootInternal(newDir, subDir) != nullptr) { reportAlways("FsZip::mount already exists: %s", - newDir.c_str()); + newDir.c_str()) return false; } ZipEntry *const entry = new ZipEntry(newDir, @@ -676,7 +676,7 @@ namespace VirtFs prepareFsPath(oldDir); if (findLast(oldDir, ".zip") == false) { - reportAlways("Called unmount without zip archive"); + reportAlways("Called unmount without zip archive") return false; } FOR_EACH (STD_VECTOR<FsEntry*>::iterator, it, mEntries) @@ -696,7 +696,7 @@ namespace VirtFs } reportAlways("VirtFs::unmountZip not exists: %s", - oldDir.c_str()); + oldDir.c_str()) return false; } @@ -706,7 +706,7 @@ namespace VirtFs prepareFsPath(oldDir); if (findLast(oldDir, ".zip") == false) { - reportAlways("Called unmount without zip archive"); + reportAlways("Called unmount without zip archive") return false; } prepareFsPath(subDir); @@ -738,7 +738,7 @@ namespace VirtFs } reportAlways("VirtFs::unmountZip not exists: %s", - oldDir.c_str()); + oldDir.c_str()) return false; } @@ -748,7 +748,7 @@ namespace VirtFs if (checkPath(fileName) == false) { reportAlways("FsDir::getRealDir invalid path: %s", - fileName.c_str()); + fileName.c_str()) return std::string(); } @@ -862,7 +862,7 @@ namespace VirtFs if (checkPath(filename) == false) { reportAlways("VirtFs::loadFile invalid path: %s", - filename.c_str()); + filename.c_str()) return nullptr; } FOR_EACH (STD_VECTOR<FsEntry*>::const_iterator, it, mEntries) diff --git a/src/fs/virtfs/fsdir.cpp b/src/fs/virtfs/fsdir.cpp index dae2a9eb1..d6beae6c4 100644 --- a/src/fs/virtfs/fsdir.cpp +++ b/src/fs/virtfs/fsdir.cpp @@ -73,7 +73,7 @@ namespace FsDir if (fd == FILEHDEFAULT) { reportAlways("VirtFs::open file open error: %s", - filename.c_str()); + filename.c_str()) return nullptr; } return new File(&funcs, fd); @@ -252,7 +252,7 @@ namespace FsDir if (checkPath(name) == false) { reportAlways("FsDir::isSymbolicLink invalid path: %s", - name.c_str()); + name.c_str()) return false; } #ifndef WIN32 @@ -286,7 +286,7 @@ namespace FsDir prepareFsPath(dirname); if (mWriteDir.empty()) { - reportAlways("FsDir::mkdir write dir is empty"); + reportAlways("FsDir::mkdir write dir is empty") return false; } return mkdir_r((mWriteDir + dirname).c_str()) != -1; @@ -297,7 +297,7 @@ namespace FsDir prepareFsPath(filename); if (mWriteDir.empty()) { - reportAlways("FsDir::remove write dir is empty"); + reportAlways("FsDir::remove write dir is empty") return false; } return ::remove((mWriteDir + filename).c_str()) != 0; @@ -326,7 +326,7 @@ namespace FsDir FILEHTYPE fd = file->mFd; if (fd == FILEHDEFAULT) { - reportAlways("FsDir::read file not opened."); + reportAlways("FsDir::read file not opened.") return 0; } #ifdef USE_FILE_FOPEN @@ -350,7 +350,7 @@ namespace FsDir FILEHTYPE fd = file->mFd; if (fd == FILEHDEFAULT) { - reportAlways("FsDir::write file not opened."); + reportAlways("FsDir::write file not opened.") return 0; } #ifdef USE_FILE_FOPEN @@ -371,14 +371,14 @@ namespace FsDir FILEHTYPE fd = file->mFd; if (fd == FILEHDEFAULT) { - reportAlways("FsDir::fileLength file not opened."); + reportAlways("FsDir::fileLength file not opened.") return 0; } #ifdef USE_FILE_FOPEN const long pos = ftell(fd); if (pos < 0) { - reportAlways("FsDir::fileLength ftell error."); + reportAlways("FsDir::fileLength ftell error.") return -1; } fseek(fd, 0, SEEK_END); @@ -389,7 +389,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") return -1; } return static_cast<int64_t>(statbuf.st_size); @@ -404,7 +404,7 @@ namespace FsDir FILEHTYPE fd = file->mFd; if (fd == FILEHDEFAULT) { - reportAlways("FsDir::tell file not opened."); + reportAlways("FsDir::tell file not opened.") return 0; } #ifdef USE_FILE_FOPEN @@ -423,7 +423,7 @@ namespace FsDir FILEHTYPE fd = file->mFd; if (fd == FILEHDEFAULT) { - reportAlways("FsDir::seek file not opened."); + reportAlways("FsDir::seek file not opened.") return 0; } const int64_t res = FILESEEK(fd, pos, SEEK_SET); @@ -440,7 +440,7 @@ namespace FsDir FILEHTYPE fd = file->mFd; if (fd == FILEHDEFAULT) { - reportAlways("FsDir::eof file not opened."); + reportAlways("FsDir::eof file not opened.") return 0; } #ifdef USE_FILE_FOPEN @@ -454,7 +454,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") return -1; } const int64_t len = static_cast<int64_t>(statbuf.st_size); @@ -475,7 +475,7 @@ namespace FsDir if (fd == FILEHDEFAULT) { reportAlways("VirtFs::loadFile file open error: %s", - filename.c_str()); + filename.c_str()) return nullptr; } @@ -488,7 +488,7 @@ namespace FsDir const long sz = ftell(fd); if (sz < 0) { - reportAlways("FsDir::fileLength ftell error."); + reportAlways("FsDir::fileLength ftell error.") if (fd != FILEHDEFAULT) FILECLOSE(fd); return nullptr; @@ -499,7 +499,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") if (fd != FILEHDEFAULT) FILECLOSE(fd); return nullptr; diff --git a/src/fs/virtfs/fsdirrwops.cpp b/src/fs/virtfs/fsdirrwops.cpp index 9906e1895..480567288 100644 --- a/src/fs/virtfs/fsdirrwops.cpp +++ b/src/fs/virtfs/fsdirrwops.cpp @@ -87,7 +87,7 @@ namespace FsDir const long curpos = ftell(fd); if (curpos < 0) { - reportAlways("FsDir::fileLength ftell error."); + reportAlways("FsDir::fileLength ftell error.") return -1; } fseek(fd, 0, SEEK_END); @@ -97,7 +97,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") len = -1; } else @@ -111,7 +111,7 @@ namespace FsDir #ifdef USE_FILE_FOPEN if (fseek(fd, curpos, SEEK_SET) < 0) { - reportAlways("FsDir::fileLength fseek error."); + reportAlways("FsDir::fileLength fseek error.") } #endif // USE_FILE_FOPEN logger->assertLog( @@ -184,7 +184,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") return CAST_S32(rc); } } @@ -220,7 +220,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") return CAST_S32(rc); } } @@ -256,7 +256,7 @@ namespace FsDir struct stat statbuf; if (fstat(fd, &statbuf) == -1) { - reportAlways("FsDir::fileLength error."); + reportAlways("FsDir::fileLength error.") return -1; } return static_cast<int64_t>(statbuf.st_size); diff --git a/src/fs/virtfs/fszip.cpp b/src/fs/virtfs/fszip.cpp index 871e3cc1a..42fe808d0 100644 --- a/src/fs/virtfs/fszip.cpp +++ b/src/fs/virtfs/fszip.cpp @@ -568,14 +568,14 @@ namespace FsZip File *openWrite(FsEntry *restrict const entry A_UNUSED, const std::string &filename A_UNUSED) { - reportAlways("VirtFs::openWrite for zip not implemented."); + reportAlways("VirtFs::openWrite for zip not implemented.") return nullptr; } File *openAppend(FsEntry *restrict const entry A_UNUSED, const std::string &filename A_UNUSED) { - reportAlways("VirtFs::openAppend for zip not implemented."); + reportAlways("VirtFs::openAppend for zip not implemented.") return nullptr; } @@ -600,7 +600,7 @@ namespace FsZip } if (buffer == nullptr) { - reportAlways("FsZip::read buffer is null"); + reportAlways("FsZip::read buffer is null") return 0; } const size_t pos = file->mPos; diff --git a/src/fs/virtfs/rwops.cpp b/src/fs/virtfs/rwops.cpp index 96c3cddd6..c6ad34eff 100644 --- a/src/fs/virtfs/rwops.cpp +++ b/src/fs/virtfs/rwops.cpp @@ -69,7 +69,7 @@ SDL_RWops *create_rwops(File *const file, if (file == nullptr) { reportAlways("VirtFs::rwops_seek: create rwops error: %s", - fname.c_str()); + fname.c_str()) } else { diff --git a/src/fs/virtfs/zipreader.cpp b/src/fs/virtfs/zipreader.cpp index e76b219b2..ec3420946 100644 --- a/src/fs/virtfs/zipreader.cpp +++ b/src/fs/virtfs/zipreader.cpp @@ -51,8 +51,8 @@ extern const char *dirSeparator; if (cnt != (sz)) \ { \ reportAlways("Error reading " msg " in file %s", \ - archiveName.c_str()); \ - delete2(header); \ + archiveName.c_str()) \ + delete2(header) \ delete [] buf; \ fclose(arcFile); \ return false; \ @@ -75,7 +75,7 @@ namespace ZipReader { if (entry == nullptr) { - reportAlways("Entry is null."); + reportAlways("Entry is null.") return false; } const std::string archiveName = entry->root; @@ -86,7 +86,7 @@ namespace ZipReader if (arcFile == nullptr) { reportAlways("Can't open zip file %s", - archiveName.c_str()); + archiveName.c_str()) return false; } uint8_t *const buf = new uint8_t[65535 + 10]; @@ -104,7 +104,7 @@ namespace ZipReader size_t cnt = 0U; // file header pointer on 0 // read file header signature - readVal(buf, 4, "zip file header"); // + 4 + readVal(buf, 4, "zip file header") // + 4 // pointer on 4 if (buf[0] == 0x50 && @@ -138,7 +138,7 @@ namespace ZipReader if (fileNameLen > 1000) { reportAlways("Error too long file name in file %s", - archiveName.c_str()); + archiveName.c_str()) delete header; delete [] buf; fclose(arcFile); @@ -148,7 +148,7 @@ namespace ZipReader swapVal16(val16) // file header pointer on 30 const uint32_t extraFieldLen = CAST_U32(val16); - readVal(buf, fileNameLen, "file name"); + readVal(buf, fileNameLen, "file name") // file header pointer on 30 + fileNameLen buf[fileNameLen] = 0; header->fileName = std::string( @@ -178,7 +178,7 @@ namespace ZipReader header->fileName.c_str()); #endif // DEBUG_ZIP dirs.push_back(header->fileName); - delete2(header); + delete2(header) } } else if (buf[0] == 0x50 && @@ -209,7 +209,7 @@ namespace ZipReader buf[1], buf[2], buf[3], - archiveName.c_str()); + archiveName.c_str()) delete [] buf; fclose(arcFile); return false; @@ -225,7 +225,7 @@ namespace ZipReader { reportAlways("Zlib error: '%s' in %s", text.c_str(), - getZlibError(err).c_str()); + getZlibError(err).c_str()) } std::string getZlibError(const int err) @@ -243,7 +243,7 @@ namespace ZipReader { if (header == nullptr) { - reportAlways("ZipReader::readCompressedFile: header is null"); + reportAlways("ZipReader::readCompressedFile: header is null") return nullptr; } FILE *restrict const arcFile = fopen( @@ -252,7 +252,7 @@ namespace ZipReader if (arcFile == nullptr) { reportAlways("Can't open zip file %s", - header->zipEntry->root.c_str()); + header->zipEntry->root.c_str()) return nullptr; } @@ -263,7 +263,7 @@ namespace ZipReader compressSize) { reportAlways("Read zip compressed file error from archive: %s", - header->zipEntry->root.c_str()); + header->zipEntry->root.c_str()) fclose(arcFile); delete [] buf; return nullptr; @@ -276,7 +276,7 @@ namespace ZipReader { if (header == nullptr) { - reportAlways("Open zip file error. header is null."); + reportAlways("Open zip file error. header is null.") return nullptr; } uint8_t *restrict const in = readCompressedFile(header); diff --git a/src/game.cpp b/src/game.cpp index dec8874cf..de782a3a7 100644 --- a/src/game.cpp +++ b/src/game.cpp @@ -311,7 +311,7 @@ static void createGuiWindows() IgnoreRecord_false, TryRemoveColors_true); } - delete2(assertListener); + delete2(assertListener) } if (config.getBoolValue("enableTradeTab")) chatWindow->addSpecialChannelTab(TRADE_CHANNEL, false); @@ -357,20 +357,20 @@ static void destroyGuiWindows() GuildManager::clear(); #endif // TMWA_SUPPORT - delete2(windowMenu); + delete2(windowMenu) delete2(localChatTab) // Need to do this first, so it can remove itself delete2(debugChatTab) delete2(tradeChatTab) delete2(battleChatTab) delete2(langChatTab) - delete2(gmChatTab); + delete2(gmChatTab) #ifdef TMWA_SUPPORT if (guildManager != nullptr && GuildManager::getEnableGuildBot()) guildManager->reload(); #endif // TMWA_SUPPORT logger->log("start deleting"); - delete2(emoteWindow); + delete2(emoteWindow) delete2(chatWindow) logger->log("end deleting"); delete2(statusWindow) @@ -388,15 +388,15 @@ static void destroyGuiWindows() delete2(emoteShortcutWindow) delete2(outfitWindow) delete2(socialWindow) - delete2(dropShortcutWindow); - delete2(spellShortcutWindow); - delete2(bankWindow); - delete2(cutInWindow); - delete2(mailWindow); - delete2(questsWindow); - delete2(whoIsOnline); - delete2(killStats); - delete2(clanWindow); + delete2(dropShortcutWindow) + delete2(spellShortcutWindow) + delete2(bankWindow) + delete2(cutInWindow) + delete2(mailWindow) + delete2(questsWindow) + delete2(whoIsOnline) + delete2(killStats) + delete2(clanWindow) } Game *Game::mInstance = nullptr; @@ -508,8 +508,8 @@ Game::~Game() delete2(mumbleManager) #endif // USE_MUMBLE - delete2(crazyMoves); - delete2(emptyBeingSlot); + delete2(crazyMoves) + delete2(emptyBeingSlot) Being::clearCache(); mInstance = nullptr; diff --git a/src/gamemodifiers.cpp b/src/gamemodifiers.cpp index f3e9dbdd0..9badf5b42 100644 --- a/src/gamemodifiers.cpp +++ b/src/gamemodifiers.cpp @@ -131,7 +131,7 @@ void GameModifiers::changeMode(unsigned *restrict const var, GameModifierListener::distributeEvent(); const std::string str = (*func)(); if (str.size() > 4) - debugMsg(str.substr(4)); + debugMsg(str.substr(4)) } const char *GameModifiers::getVarItem(const char *const *const arr, diff --git a/src/graphicsmanager.cpp b/src/graphicsmanager.cpp index 42bbc73a0..599466aba 100644 --- a/src/graphicsmanager.cpp +++ b/src/graphicsmanager.cpp @@ -105,8 +105,10 @@ PRAGMA48(GCC diagnostic pop) #define GL_MAX_RENDERBUFFER_SIZE 0x84E8 #endif // GL_MAX_RENDERBUFFER_SIZE #define useCompression(name) \ - OpenGLImageHelper::setInternalTextureType(name); \ - logger->log("using " #name " texture compression"); + { \ + OpenGLImageHelper::setInternalTextureType(name); \ + logger->log("using " #name " texture compression"); \ + } #endif // USE_OPENGL GraphicsManager graphicsManager; @@ -378,7 +380,7 @@ void GraphicsManager::createRenderers() case RENDER_SDL2_DEFAULT: RENDER_SDL2_DEFAULT_INIT break; - }; + } mUseAtlases = (useOpenGL == RENDER_NORMAL_OPENGL || useOpenGL == RENDER_SAFE_OPENGL || useOpenGL == RENDER_MODERN_OPENGL || @@ -440,11 +442,11 @@ void GraphicsManager::createRenderers() void GraphicsManager::deleteRenderers() { - delete2(mainGraphics); + delete2(mainGraphics) if (imageHelper != surfaceImageHelper) delete surfaceImageHelper; surfaceImageHelper = nullptr; - delete2(imageHelper); + delete2(imageHelper) } void GraphicsManager::setVideoMode() @@ -641,7 +643,7 @@ void GraphicsManager::updateExtensions() logger->log1("opengl extensions: "); if (checkGLVersion(3, 0)) { // get extensions in new way - assignFunction2(glGetStringi, "glGetStringi"); + assignFunction2(glGetStringi, "glGetStringi") std::string extList; int num = 0; glGetIntegerv(GL_NUM_EXTENSIONS, &num); @@ -669,7 +671,7 @@ void GraphicsManager::updateExtensions() void GraphicsManager::updatePlanformExtensions() { SDL_SysWMinfo info; - SDL_VERSION(&info.version); + SDL_VERSION(&info.version) if (SDL::getWindowWMInfo(mainGraphics->getWindow(), &info)) { #ifdef WIN32 @@ -777,7 +779,7 @@ void GraphicsManager::updateTextureCompressionFormat() const if (compressionFormat == 1) { delete []formats; - useCompression(GL_COMPRESSED_RGBA_S3TC_DXT5_EXT); + useCompression(GL_COMPRESSED_RGBA_S3TC_DXT5_EXT) return; } break; @@ -785,7 +787,7 @@ void GraphicsManager::updateTextureCompressionFormat() const if (compressionFormat == 2) { delete []formats; - useCompression(GL_COMPRESSED_RGBA_FXT1_3DFX); + useCompression(GL_COMPRESSED_RGBA_FXT1_3DFX) return; } break; @@ -793,7 +795,7 @@ void GraphicsManager::updateTextureCompressionFormat() const if (compressionFormat == 4) { delete []formats; - useCompression(GL_COMPRESSED_RGBA_BPTC_UNORM_ARB); + useCompression(GL_COMPRESSED_RGBA_BPTC_UNORM_ARB) return; } break; @@ -804,7 +806,7 @@ void GraphicsManager::updateTextureCompressionFormat() const if (compressionFormat == 3) { delete []formats; - useCompression(GL_COMPRESSED_RGBA_ARB); + useCompression(GL_COMPRESSED_RGBA_ARB) return; } @@ -813,7 +815,7 @@ void GraphicsManager::updateTextureCompressionFormat() const && supportExtension("GL_ARB_texture_compression_bptc")) { delete []formats; - useCompression(GL_COMPRESSED_RGBA_BPTC_UNORM_ARB); + useCompression(GL_COMPRESSED_RGBA_BPTC_UNORM_ARB) return; } } @@ -1023,10 +1025,10 @@ void GraphicsManager::initOpenGLFunctions() if (is10 && (is33 || supportExtension("GL_ARB_sampler_objects"))) { logger->log1("found GL_ARB_sampler_objects"); - assignFunction(glGenSamplers); - assignFunction(glDeleteSamplers); - assignFunction(glBindSampler); - assignFunction(glSamplerParameteri); + assignFunction(glGenSamplers) + assignFunction(glDeleteSamplers) + assignFunction(glBindSampler) + assignFunction(glSamplerParameteri) if (isGLNotNull(mglGenSamplers) && config.getBoolValue("useTextureSampler")) { @@ -1076,19 +1078,19 @@ void GraphicsManager::initOpenGLFunctions() if (is45) { logger->log1("found GL_EXT_direct_state_access"); - assignFunction(glTextureSubImage2D); + assignFunction(glTextureSubImage2D) } else if (supportExtension("GL_EXT_direct_state_access")) { logger->log1("found GL_EXT_direct_state_access"); assignFunctionEmu2(glTextureSubImage2DEXT, - "glTextureSubImage2DEXT"); + "glTextureSubImage2DEXT") } else if (supportExtension("GL_ARB_direct_state_access")) { logger->log1("found GL_ARB_direct_state_access"); logger->log1("GL_EXT_direct_state_access not found"); - assignFunction(glTextureSubImage2D); + assignFunction(glTextureSubImage2D) } else { @@ -1107,7 +1109,7 @@ void GraphicsManager::initOpenGLFunctions() if (is12 && (is42 || supportExtension("GL_ARB_texture_storage"))) { logger->log1("found GL_ARB_texture_storage"); - assignFunction(glTexStorage2D); + assignFunction(glTexStorage2D) } else { @@ -1117,7 +1119,7 @@ void GraphicsManager::initOpenGLFunctions() if (is13 || supportExtension("GL_ARB_multitexture")) { logger->log1("found GL_ARB_multitexture or OpenGL 1.3"); - assignFunction(glActiveTexture); + assignFunction(glActiveTexture) } else { @@ -1128,7 +1130,7 @@ void GraphicsManager::initOpenGLFunctions() if (is20 || supportExtension("GL_ARB_explicit_attrib_location")) { logger->log1("found GL_ARB_explicit_attrib_location or OpenGL 2.0"); - assignFunction(glBindAttribLocation); + assignFunction(glBindAttribLocation) } else { @@ -1138,29 +1140,29 @@ void GraphicsManager::initOpenGLFunctions() if (is30 || supportExtension("GL_ARB_framebuffer_object")) { // frame buffer supported logger->log1("found GL_ARB_framebuffer_object"); - assignFunction(glGenRenderbuffers); - assignFunction(glBindRenderbuffer); - assignFunction(glRenderbufferStorage); - assignFunction(glGenFramebuffers); - assignFunction(glBindFramebuffer); - assignFunction(glFramebufferTexture2D); - assignFunction(glFramebufferRenderbuffer); - assignFunction(glDeleteFramebuffers); - assignFunction(glDeleteRenderbuffers); - assignFunction(glCheckFramebufferStatus); + assignFunction(glGenRenderbuffers) + assignFunction(glBindRenderbuffer) + assignFunction(glRenderbufferStorage) + assignFunction(glGenFramebuffers) + assignFunction(glBindFramebuffer) + assignFunction(glFramebufferTexture2D) + assignFunction(glFramebufferRenderbuffer) + assignFunction(glDeleteFramebuffers) + assignFunction(glDeleteRenderbuffers) + assignFunction(glCheckFramebufferStatus) } else if (supportExtension("GL_EXT_framebuffer_object")) { // old frame buffer extension logger->log1("found GL_EXT_framebuffer_object"); - assignFunctionEXT(glGenRenderbuffers); - assignFunctionEXT(glBindRenderbuffer); - assignFunctionEXT(glRenderbufferStorage); - assignFunctionEXT(glGenFramebuffers); - assignFunctionEXT(glBindFramebuffer); - assignFunctionEXT(glFramebufferTexture2D); - assignFunctionEXT(glFramebufferRenderbuffer); - assignFunctionEXT(glDeleteFramebuffers); - assignFunctionEXT(glDeleteRenderbuffers); + assignFunctionEXT(glGenRenderbuffers) + assignFunctionEXT(glBindRenderbuffer) + assignFunctionEXT(glRenderbufferStorage) + assignFunctionEXT(glGenFramebuffers) + assignFunctionEXT(glBindFramebuffer) + assignFunctionEXT(glFramebufferTexture2D) + assignFunctionEXT(glFramebufferRenderbuffer) + assignFunctionEXT(glDeleteFramebuffers) + assignFunctionEXT(glDeleteRenderbuffers) } else { // no frame buffer support @@ -1173,18 +1175,18 @@ void GraphicsManager::initOpenGLFunctions() if (is43 || supportExtension("GL_KHR_debug")) { logger->log1("found GL_KHR_debug"); - assignFunction(glDebugMessageControl); - assignFunction(glDebugMessageCallback); - assignFunction(glPushDebugGroup); - assignFunction(glPopDebugGroup); - assignFunction(glObjectLabel); + assignFunction(glDebugMessageControl) + assignFunction(glDebugMessageCallback) + assignFunction(glPushDebugGroup) + assignFunction(glPopDebugGroup) + assignFunction(glObjectLabel) mSupportDebug = 2; } else if (supportExtension("GL_ARB_debug_output")) { logger->log1("found GL_ARB_debug_output"); - assignFunctionARB(glDebugMessageControl); - assignFunctionARB(glDebugMessageCallback); + assignFunctionARB(glDebugMessageControl) + assignFunctionARB(glDebugMessageCallback) mSupportDebug = 1; } else @@ -1196,7 +1198,7 @@ void GraphicsManager::initOpenGLFunctions() if (supportExtension("GL_GREMEDY_frame_terminator")) { logger->log1("found GL_GREMEDY_frame_terminator"); - assignFunction2(glFrameTerminator, "glFrameTerminatorGREMEDY"); + assignFunction2(glFrameTerminator, "glFrameTerminatorGREMEDY") } else { @@ -1205,10 +1207,10 @@ void GraphicsManager::initOpenGLFunctions() if (is44 || supportExtension("GL_EXT_debug_label")) { logger->log1("found GL_EXT_debug_label"); - assignFunction2(glLabelObject, "glObjectLabel"); + assignFunction2(glLabelObject, "glObjectLabel") if (isGLNull(mglLabelObject)) - assignFunctionEXT(glLabelObject); - assignFunctionEXT(glGetObjectLabel); + assignFunctionEXT(glLabelObject) + assignFunctionEXT(glGetObjectLabel) } else { @@ -1217,7 +1219,7 @@ void GraphicsManager::initOpenGLFunctions() if (supportExtension("GL_GREMEDY_string_marker")) { logger->log1("found GL_GREMEDY_string_marker"); - assignFunction2(glPushGroupMarker, "glStringMarkerGREMEDY"); + assignFunction2(glPushGroupMarker, "glStringMarkerGREMEDY") } else { @@ -1226,9 +1228,9 @@ void GraphicsManager::initOpenGLFunctions() if (supportExtension("GL_EXT_debug_marker")) { logger->log1("found GL_EXT_debug_marker"); - assignFunctionEXT(glInsertEventMarker); - assignFunctionEXT(glPushGroupMarker); - assignFunctionEXT(glPopGroupMarker); + assignFunctionEXT(glInsertEventMarker) + assignFunctionEXT(glPushGroupMarker) + assignFunctionEXT(glPopGroupMarker) } else { @@ -1237,12 +1239,12 @@ void GraphicsManager::initOpenGLFunctions() if (is15 && (is30 || supportExtension("GL_EXT_timer_query"))) { logger->log1("found GL_EXT_timer_query"); - assignFunction(glGenQueries); - assignFunction(glBeginQuery); - assignFunction(glEndQuery); - assignFunction(glDeleteQueries); - assignFunction(glGetQueryObjectiv); - assignFunctionEXT(glGetQueryObjectui64v); + assignFunction(glGenQueries) + assignFunction(glBeginQuery) + assignFunction(glEndQuery) + assignFunction(glDeleteQueries) + assignFunction(glGetQueryObjectiv) + assignFunctionEXT(glGetQueryObjectui64v) } else { @@ -1251,7 +1253,7 @@ void GraphicsManager::initOpenGLFunctions() if (is20 && (is43 || supportExtension("GL_ARB_invalidate_subdata"))) { logger->log1("found GL_ARB_invalidate_subdata"); - assignFunction(glInvalidateTexImage); + assignFunction(glInvalidateTexImage) } else { @@ -1260,13 +1262,13 @@ void GraphicsManager::initOpenGLFunctions() if (is21 && (is30 || supportExtension("GL_ARB_vertex_array_object"))) { logger->log1("found GL_ARB_vertex_array_object"); - assignFunction(glGenVertexArrays); - assignFunction(glBindVertexArray); - assignFunction(glDeleteVertexArrays); - assignFunction(glVertexAttribPointer); - assignFunction(glEnableVertexAttribArray); - assignFunction(glDisableVertexAttribArray); - assignFunction(glVertexAttribIPointer); + assignFunction(glGenVertexArrays) + assignFunction(glBindVertexArray) + assignFunction(glDeleteVertexArrays) + assignFunction(glVertexAttribPointer) + assignFunction(glEnableVertexAttribArray) + assignFunction(glDisableVertexAttribArray) + assignFunction(glVertexAttribIPointer) } else { @@ -1275,11 +1277,11 @@ void GraphicsManager::initOpenGLFunctions() } if (is20 || supportExtension("GL_ARB_vertex_buffer_object")) { - assignFunction(glGenBuffers); - assignFunction(glDeleteBuffers); - assignFunction(glBindBuffer); - assignFunction(glBufferData); - assignFunction(glIsBuffer); + assignFunction(glGenBuffers) + assignFunction(glDeleteBuffers) + assignFunction(glBindBuffer) + assignFunction(glBufferData) + assignFunction(glIsBuffer) } else { @@ -1289,7 +1291,7 @@ void GraphicsManager::initOpenGLFunctions() if (is43 || supportExtension("GL_ARB_copy_image")) { logger->log1("found GL_ARB_copy_image"); - assignFunction(glCopyImageSubData); + assignFunction(glCopyImageSubData) } else { @@ -1298,8 +1300,8 @@ void GraphicsManager::initOpenGLFunctions() if (is44 || supportExtension("GL_ARB_clear_texture")) { logger->log1("found GL_ARB_clear_texture"); - assignFunction(glClearTexImage); - assignFunction(glClearTexSubImage); + assignFunction(glClearTexImage) + assignFunction(glClearTexSubImage) } else { @@ -1308,35 +1310,35 @@ void GraphicsManager::initOpenGLFunctions() if (is20 || supportExtension("GL_ARB_shader_objects")) { logger->log1("found GL_ARB_shader_objects"); - assignFunction(glCreateShader); - assignFunction(glDeleteShader); - assignFunction(glGetShaderiv); - assignFunction(glGetShaderInfoLog); - assignFunction(glGetShaderSource); - assignFunction(glShaderSource); - assignFunction(glCompileShader); - assignFunction(glLinkProgram); - assignFunction(glGetProgramInfoLog); - assignFunction(glDeleteProgram); - assignFunction(glCreateProgram); - assignFunction(glAttachShader); - assignFunction(glDetachShader); - assignFunction(glGetAttachedShaders); - assignFunction(glGetUniformLocation); - assignFunction(glGetActiveUniform); - assignFunction(glGetProgramiv); - assignFunction(glUseProgram); - assignFunction(glValidateProgram); - assignFunction(glGetAttribLocation); - assignFunction(glUniform1f); - assignFunction(glUniform2f); - assignFunction(glUniform3f); - assignFunction(glUniform4f); + assignFunction(glCreateShader) + assignFunction(glDeleteShader) + assignFunction(glGetShaderiv) + assignFunction(glGetShaderInfoLog) + assignFunction(glGetShaderSource) + assignFunction(glShaderSource) + assignFunction(glCompileShader) + assignFunction(glLinkProgram) + assignFunction(glGetProgramInfoLog) + assignFunction(glDeleteProgram) + assignFunction(glCreateProgram) + assignFunction(glAttachShader) + assignFunction(glDetachShader) + assignFunction(glGetAttachedShaders) + assignFunction(glGetUniformLocation) + assignFunction(glGetActiveUniform) + assignFunction(glGetProgramiv) + assignFunction(glUseProgram) + assignFunction(glValidateProgram) + assignFunction(glGetAttribLocation) + assignFunction(glUniform1f) + assignFunction(glUniform2f) + assignFunction(glUniform3f) + assignFunction(glUniform4f) if (is30 || supportExtension("GL_EXT_gpu_shader4")) { logger->log1("found GL_EXT_gpu_shader4"); - assignFunction(glBindFragDataLocation); + assignFunction(glBindFragDataLocation) } else { @@ -1346,10 +1348,10 @@ void GraphicsManager::initOpenGLFunctions() if (is41 || supportExtension("GL_ARB_separate_shader_objects")) { logger->log1("found GL_ARB_separate_shader_objects"); - assignFunction(glProgramUniform1f); - assignFunction(glProgramUniform2f); - assignFunction(glProgramUniform3f); - assignFunction(glProgramUniform4f); + assignFunction(glProgramUniform1f) + assignFunction(glProgramUniform2f) + assignFunction(glProgramUniform3f) + assignFunction(glProgramUniform4f) } else { @@ -1358,10 +1360,10 @@ void GraphicsManager::initOpenGLFunctions() if (is43 || supportExtension("GL_ARB_vertex_attrib_binding")) { logger->log1("found GL_ARB_vertex_attrib_binding"); - assignFunction(glBindVertexBuffer); - assignFunction(glVertexAttribBinding); - assignFunction(glVertexAttribFormat); - assignFunction(glVertexAttribIFormat); + assignFunction(glBindVertexBuffer) + assignFunction(glVertexAttribBinding) + assignFunction(glVertexAttribFormat) + assignFunction(glVertexAttribIFormat) } else { @@ -1371,7 +1373,7 @@ void GraphicsManager::initOpenGLFunctions() if (is44 || supportExtension("GL_ARB_multi_bind")) { logger->log1("found GL_ARB_multi_bind"); - assignFunction(glBindVertexBuffers); + assignFunction(glBindVertexBuffers) } else { @@ -1385,7 +1387,7 @@ void GraphicsManager::initOpenGLFunctions() } #ifdef WIN32 - assignFunctionARB(wglGetExtensionsString); + assignFunctionARB(wglGetExtensionsString) #endif // WIN32 #endif // __native_client__ } @@ -1653,7 +1655,7 @@ void GraphicsManager::updateDebugLog() const void GraphicsManager::detectPixelSize() { SDL_SysWMinfo info; - SDL_VERSION(&info.version); + SDL_VERSION(&info.version) if (SDL::getWindowWMInfo(mainGraphics->getWindow(), &info)) { #ifdef WIN32 diff --git a/src/gui/fonts/font.cpp b/src/gui/fonts/font.cpp index 9d09d7855..5ce6b88d0 100644 --- a/src/gui/fonts/font.cpp +++ b/src/gui/fonts/font.cpp @@ -124,7 +124,7 @@ Font::Font(std::string filename, { reportAlways("Error: requested load font %s with size %d", filename.c_str(), - size); + size) size = 4; } @@ -151,7 +151,7 @@ Font::Font(std::string filename, { #ifdef UNITTESTS reportAlways("Font load failed %s", - filename.c_str()); + filename.c_str()) #endif // UNITTESTS logger->error("Font::Font: " + std::string(SDL_GetError())); @@ -205,7 +205,7 @@ TTF_Font *Font::openFont(const char *const name, #ifndef UNITTESTS // +++ in future need trigger assert in unit tests here too reportAlways("Font::openFont font not exists: %s", - path.c_str()); + path.c_str()) #endif // UNITTESTS return nullptr; } @@ -489,7 +489,7 @@ void Font::insertChunk(TextChunk *const chunk) chunk->color, chunk->color2)); if (i != search.end()) { - delete2(chunk->img); + delete2(chunk->img) return; } diff --git a/src/gui/fonts/textchunk.cpp b/src/gui/fonts/textchunk.cpp index 9280b9241..75b9c9626 100644 --- a/src/gui/fonts/textchunk.cpp +++ b/src/gui/fonts/textchunk.cpp @@ -81,7 +81,7 @@ TextChunk::TextChunk(const std::string &restrict text0, TextChunk::~TextChunk() { - delete2(img); + delete2(img) #ifdef UNITTESTS textChunkCnt --; #endif // UNITTESTS @@ -198,6 +198,6 @@ void TextChunk::deleteImage() } else { - delete2(img); + delete2(img) } } diff --git a/src/gui/gui.cpp b/src/gui/gui.cpp index be516ebe9..4e9133188 100644 --- a/src/gui/gui.cpp +++ b/src/gui/gui.cpp @@ -267,7 +267,7 @@ void Gui::postInit(Graphics *const graphics) Gui::~Gui() { config.removeListeners(mConfigListener); - delete2(mConfigListener); + delete2(mConfigListener) if (mMouseCursors != nullptr) { @@ -283,16 +283,16 @@ Gui::~Gui() Window::setWindowContainer(nullptr); delete top; - delete2(mGuiFont); - delete2(boldFont); - delete2(mHelpFont); - delete2(mSecureFont); - delete2(mInfoParticleFont); - delete2(mNpcFont); - delete2(guiInput); - delete2(theme); + delete2(mGuiFont) + delete2(boldFont) + delete2(mHelpFont) + delete2(mSecureFont) + delete2(mInfoParticleFont) + delete2(mNpcFont) + delete2(guiInput) + delete2(theme) - delete2(mFocusHandler); + delete2(mFocusHandler) Label::finalCleanup(); Tab::finalCleanup(); Widget::cleanGlobalFont(); diff --git a/src/gui/models/colormodel.cpp b/src/gui/models/colormodel.cpp index 694fecff0..e30458750 100644 --- a/src/gui/models/colormodel.cpp +++ b/src/gui/models/colormodel.cpp @@ -66,7 +66,7 @@ void ColorModel::add(const std::string &name, const Color *const color1, #define addColor(name, color) \ model->add(name, &widget->getThemeColor(ThemeColorId::color, 255U), \ - &widget->getThemeColor(ThemeColorId::color##_OUTLINE, 255U)); + &widget->getThemeColor(ThemeColorId::color##_OUTLINE, 255U)) ColorModel *ColorModel::createDefault(const Widget2 *const widget) { diff --git a/src/gui/popups/popupmenu.cpp b/src/gui/popups/popupmenu.cpp index 1d4745110..c795a0384 100644 --- a/src/gui/popups/popupmenu.cpp +++ b/src/gui/popups/popupmenu.cpp @@ -1546,7 +1546,7 @@ void PopupMenu::handleLink(const std::string &link, // Unknown actions else if (link != "cancel") { - reportAlways("PopupMenu: Warning, unknown action '%s'", link.c_str()); + reportAlways("PopupMenu: Warning, unknown action '%s'", link.c_str()) } if (!mAllowCleanMenu) diff --git a/src/gui/popups/statuspopup.cpp b/src/gui/popups/statuspopup.cpp index 863dd38ef..dd2b317ab 100644 --- a/src/gui/popups/statuspopup.cpp +++ b/src/gui/popups/statuspopup.cpp @@ -59,27 +59,27 @@ void StatusPopup::postInit() Popup::postInit(); const int fontHeight = getFont()->getHeight(); int y = 0; - addLabel(0); - addLabel(1); - addLabel(2); - addLabel(3); + addLabel(0) + addLabel(1) + addLabel(2) + addLabel(3) y += 4; - addLabel(4); - addLabel(5); - addLabel(9); - addLabel(10); + addLabel(4) + addLabel(5) + addLabel(9) + addLabel(10) y += 4; - addLabel(6); - addLabel(7); + addLabel(6) + addLabel(7) y += 4; - addLabel(8); + addLabel(8) y += 4; - addLabel(12); - addLabel(13); - addLabel(14); - addLabel(15); + addLabel(12) + addLabel(13) + addLabel(14) + addLabel(15) y += 4; - addLabel(11); + addLabel(11) } StatusPopup::~StatusPopup() diff --git a/src/gui/skin.cpp b/src/gui/skin.cpp index 65f7a1b41..2797ad0d1 100644 --- a/src/gui/skin.cpp +++ b/src/gui/skin.cpp @@ -96,8 +96,8 @@ Skin::~Skin() mStickyImageDown = nullptr; } - delete2(mOptions); - delete2(mBorder); + delete2(mOptions) + delete2(mBorder) } void Skin::updateAlpha(const float minimumOpacityAllowed) diff --git a/src/gui/theme.cpp b/src/gui/theme.cpp index c6d1e1bd1..4954e0d8c 100644 --- a/src/gui/theme.cpp +++ b/src/gui/theme.cpp @@ -891,7 +891,7 @@ void Theme::loadRect(ImageRect &image, Theme::getThemePath()); if (skin != nullptr) { - loadGrid(); + loadGrid() unload(skin); } } @@ -907,7 +907,7 @@ Skin *Theme::loadSkinRect(ImageRect &image, true, Theme::getThemePath()); if (skin != nullptr) - loadGrid(); + loadGrid() return skin; } diff --git a/src/gui/widgets/attrs/attrdisplay.cpp b/src/gui/widgets/attrs/attrdisplay.cpp index fe5c1f3e3..5ab26c02b 100644 --- a/src/gui/widgets/attrs/attrdisplay.cpp +++ b/src/gui/widgets/attrs/attrdisplay.cpp @@ -52,7 +52,7 @@ AttrDisplay::AttrDisplay(const Widget2 *const widget, AttrDisplay::~AttrDisplay() { - delete2(mLayout); + delete2(mLayout) } std::string AttrDisplay::update() diff --git a/src/gui/widgets/basiccontainer.cpp b/src/gui/widgets/basiccontainer.cpp index 92611687d..95fe9e1ef 100644 --- a/src/gui/widgets/basiccontainer.cpp +++ b/src/gui/widgets/basiccontainer.cpp @@ -82,7 +82,7 @@ void BasicContainer::moveToTop(Widget *const widget) restrict2 { if (widget == nullptr) { - reportAlways("BasicContainer::moveToTop empty widget."); + reportAlways("BasicContainer::moveToTop empty widget.") return; } @@ -110,7 +110,7 @@ void BasicContainer::moveToBottom(Widget *const widget) restrict2 { if (widget == nullptr) { - reportAlways("BasicContainer::moveToBottom empty widget."); + reportAlways("BasicContainer::moveToBottom empty widget.") return; } const WidgetListIterator iter = std::find(mWidgets.begin(), diff --git a/src/gui/widgets/browserbox.cpp b/src/gui/widgets/browserbox.cpp index 7354dbabf..82039de0c 100644 --- a/src/gui/widgets/browserbox.cpp +++ b/src/gui/widgets/browserbox.cpp @@ -413,7 +413,7 @@ void BrowserBox::draw(Graphics *const graphics) setHeight(mHeight); mUpdateTime = cur_time; if (mDimension.width != mWidth) - reportAlways("browserbox resize in draw"); + reportAlways("browserbox resize in draw") } if (mOpaque == Opaque_true) diff --git a/src/gui/widgets/button.cpp b/src/gui/widgets/button.cpp index 34112e5a5..840171207 100644 --- a/src/gui/widgets/button.cpp +++ b/src/gui/widgets/button.cpp @@ -346,7 +346,7 @@ Button::~Button() for (int mode = 0; mode < BUTTON_COUNT; mode ++) theme->unload(mSkin[mode]); } - delete2(mVertexes2); + delete2(mVertexes2) if (mImageSet != nullptr) { mImageSet->decRef(); diff --git a/src/gui/widgets/emotepage.cpp b/src/gui/widgets/emotepage.cpp index d10acc705..ffd04302d 100644 --- a/src/gui/widgets/emotepage.cpp +++ b/src/gui/widgets/emotepage.cpp @@ -60,7 +60,7 @@ EmotePage::~EmotePage() mEmotes->decRef(); mEmotes = nullptr; } - delete2(mVertexes); + delete2(mVertexes) } void EmotePage::draw(Graphics *const graphics) diff --git a/src/gui/widgets/guitable.cpp b/src/gui/widgets/guitable.cpp index 7dc8c288a..2db154dc9 100644 --- a/src/gui/widgets/guitable.cpp +++ b/src/gui/widgets/guitable.cpp @@ -73,7 +73,7 @@ GuiTable::~GuiTable() gui->removeDragged(this); uninstallActionListeners(); - delete2(mModel); + delete2(mModel) } const TableModel *GuiTable::getModel() const diff --git a/src/gui/widgets/itemcontainer.cpp b/src/gui/widgets/itemcontainer.cpp index 69c816a68..fba09fddb 100644 --- a/src/gui/widgets/itemcontainer.cpp +++ b/src/gui/widgets/itemcontainer.cpp @@ -297,7 +297,7 @@ ItemContainer::~ItemContainer() theme->unload(mSkin); delete []mShowMatrix; - delete2(mVertexes); + delete2(mVertexes) } void ItemContainer::logic() @@ -745,7 +745,7 @@ void ItemContainer::mouseReleased(MouseEvent &event) case SEL_SELECTED: default: break; - }; + } if (dragDrop.isEmpty()) { diff --git a/src/gui/widgets/popup.cpp b/src/gui/widgets/popup.cpp index c676de26e..05487b7be 100644 --- a/src/gui/widgets/popup.cpp +++ b/src/gui/widgets/popup.cpp @@ -84,7 +84,7 @@ Popup::~Popup() { logger->log("Popup::~Popup(\"%s\")", mPopupName.c_str()); - delete2(mVertexes); + delete2(mVertexes) if (mSkin != nullptr) { diff --git a/src/gui/widgets/progressbar.cpp b/src/gui/widgets/progressbar.cpp index 46988133b..05933fe89 100644 --- a/src/gui/widgets/progressbar.cpp +++ b/src/gui/widgets/progressbar.cpp @@ -120,7 +120,7 @@ ProgressBar::~ProgressBar() mSkin = nullptr; } Theme::unloadRect(mFillRect, 0, 8); - delete2(mVertexes); + delete2(mVertexes) mTextChunk.deleteImage(); } diff --git a/src/gui/widgets/progressindicator.cpp b/src/gui/widgets/progressindicator.cpp index f87c3cb09..30e7da5e8 100644 --- a/src/gui/widgets/progressindicator.cpp +++ b/src/gui/widgets/progressindicator.cpp @@ -60,7 +60,7 @@ ProgressIndicator::~ProgressIndicator() if (gui != nullptr) gui->removeDragged(this); - delete2(mIndicator); + delete2(mIndicator) } void ProgressIndicator::logic() diff --git a/src/gui/widgets/scrollarea.cpp b/src/gui/widgets/scrollarea.cpp index cce45deee..f313ee400 100644 --- a/src/gui/widgets/scrollarea.cpp +++ b/src/gui/widgets/scrollarea.cpp @@ -172,8 +172,8 @@ ScrollArea::~ScrollArea() } } - delete2(mVertexes); - delete2(mVertexes2); + delete2(mVertexes) + delete2(mVertexes2) setContent(nullptr); } diff --git a/src/gui/widgets/selldialog.cpp b/src/gui/widgets/selldialog.cpp index dc612c0ac..b1bddd32b 100644 --- a/src/gui/widgets/selldialog.cpp +++ b/src/gui/widgets/selldialog.cpp @@ -221,7 +221,7 @@ void SellDialog::postInit() SellDialog::~SellDialog() { - delete2(mShopItems); + delete2(mShopItems) instances.remove(this); } diff --git a/src/gui/widgets/shortcutcontainer.cpp b/src/gui/widgets/shortcutcontainer.cpp index 5eb4ea46f..14a774bb7 100644 --- a/src/gui/widgets/shortcutcontainer.cpp +++ b/src/gui/widgets/shortcutcontainer.cpp @@ -91,7 +91,7 @@ ShortcutContainer::~ShortcutContainer() if (gui != nullptr) gui->removeDragged(this); - delete2(mVertexes); + delete2(mVertexes) } void ShortcutContainer::widgetResized(const Event &event A_UNUSED) diff --git a/src/gui/widgets/slider.cpp b/src/gui/widgets/slider.cpp index 2c4c6c212..4caaa6a45 100644 --- a/src/gui/widgets/slider.cpp +++ b/src/gui/widgets/slider.cpp @@ -135,7 +135,7 @@ Slider::~Slider() if (gui != nullptr) gui->removeDragged(this); - delete2(mVertexes); + delete2(mVertexes) mInstances--; if (mInstances == 0) { diff --git a/src/gui/widgets/staticbrowserbox.cpp b/src/gui/widgets/staticbrowserbox.cpp index c4a705aa9..13d3bfb64 100644 --- a/src/gui/widgets/staticbrowserbox.cpp +++ b/src/gui/widgets/staticbrowserbox.cpp @@ -362,7 +362,7 @@ void StaticBrowserBox::draw(Graphics *const graphics) updateHeight(); reportAlways("browserbox resize in draw: %d, %d", mDimension.width, - mWidth); + mWidth) } if (mOpaque == Opaque_true) diff --git a/src/gui/widgets/tabbedarea.cpp b/src/gui/widgets/tabbedarea.cpp index 59882966c..693a53e10 100644 --- a/src/gui/widgets/tabbedarea.cpp +++ b/src/gui/widgets/tabbedarea.cpp @@ -136,14 +136,14 @@ TabbedArea::~TabbedArea() remove(mTabContainer); remove(mWidgetContainer); - delete2(mTabContainer); - delete2(mWidgetContainer); + delete2(mTabContainer) + delete2(mWidgetContainer) for (size_t i = 0, sz = mTabsToDelete.size(); i < sz; i++) delete2(mTabsToDelete[i]) - delete2(mArrowButton[0]); - delete2(mArrowButton[1]); + delete2(mArrowButton[0]) + delete2(mArrowButton[1]) } void TabbedArea::enableScrollButtons(const bool enable) diff --git a/src/gui/widgets/tabs/chat/chattab.cpp b/src/gui/widgets/tabs/chat/chattab.cpp index 8d3ccafbb..67761a21c 100644 --- a/src/gui/widgets/tabs/chat/chattab.cpp +++ b/src/gui/widgets/tabs/chat/chattab.cpp @@ -103,8 +103,8 @@ ChatTab::~ChatTab() if (chatWindow != nullptr) chatWindow->removeTab(this); - delete2(mTextOutput); - delete2(mScrollArea); + delete2(mTextOutput) + delete2(mScrollArea) } void ChatTab::chatLog(std::string line, diff --git a/src/gui/widgets/tabs/mapdebugtab.cpp b/src/gui/widgets/tabs/mapdebugtab.cpp index 6b133e4bb..36a96084d 100644 --- a/src/gui/widgets/tabs/mapdebugtab.cpp +++ b/src/gui/widgets/tabs/mapdebugtab.cpp @@ -125,7 +125,7 @@ MapDebugTab::MapDebugTab(const Widget2 *const widget) : // TRANSLATORS: debug window label mFPSText = _("%d FPS (SDL2 default)"); break; - }; + } #else // USE_OPENGL // TRANSLATORS: debug window label diff --git a/src/gui/widgets/tabs/setup_audio.cpp b/src/gui/widgets/tabs/setup_audio.cpp index 2a9d63eba..e9617a178 100644 --- a/src/gui/widgets/tabs/setup_audio.cpp +++ b/src/gui/widgets/tabs/setup_audio.cpp @@ -205,8 +205,8 @@ Setup_Audio::Setup_Audio(const Widget2 *const widget) : Setup_Audio::~Setup_Audio() { - delete2(mSoundModel); - delete2(mChannelsList); + delete2(mSoundModel) + delete2(mChannelsList) } void Setup_Audio::apply() diff --git a/src/gui/widgets/tabs/setup_input.cpp b/src/gui/widgets/tabs/setup_input.cpp index 49b31c5fe..835e6075d 100644 --- a/src/gui/widgets/tabs/setup_input.cpp +++ b/src/gui/widgets/tabs/setup_input.cpp @@ -138,14 +138,14 @@ Setup_Input::Setup_Input(const Widget2 *const widget) : Setup_Input::~Setup_Input() { - delete2(mKeyList); - delete2(mKeyListModel); - delete2(mAssignKeyButton); - delete2(mUnassignKeyButton); - delete2(mResetKeysButton); + delete2(mKeyList) + delete2(mKeyListModel) + delete2(mAssignKeyButton) + delete2(mUnassignKeyButton) + delete2(mResetKeysButton) delete [] mActionDataSize; mActionDataSize = nullptr; - delete2(mScrollArea); + delete2(mScrollArea) } void Setup_Input::apply() diff --git a/src/gui/widgets/tabs/setup_joystick.cpp b/src/gui/widgets/tabs/setup_joystick.cpp index e3dd95caf..b2bf2c3a7 100644 --- a/src/gui/widgets/tabs/setup_joystick.cpp +++ b/src/gui/widgets/tabs/setup_joystick.cpp @@ -105,7 +105,7 @@ Setup_Joystick::Setup_Joystick(const Widget2 *const widget) : Setup_Joystick::~Setup_Joystick() { - delete2(mNamesModel); + delete2(mNamesModel) } void Setup_Joystick::action(const ActionEvent &event) diff --git a/src/gui/widgets/tabs/setup_misc.cpp b/src/gui/widgets/tabs/setup_misc.cpp index 408184311..713ad7db9 100644 --- a/src/gui/widgets/tabs/setup_misc.cpp +++ b/src/gui/widgets/tabs/setup_misc.cpp @@ -572,11 +572,11 @@ Setup_Misc::Setup_Misc(const Widget2 *const widget) : Setup_Misc::~Setup_Misc() { - delete2(mProxyTypeList); - delete2(mShortcutsList); - delete2(mDensityList); + delete2(mProxyTypeList) + delete2(mShortcutsList) + delete2(mDensityList) #ifdef USE_SDL2 - delete2(mSdlLogList); + delete2(mSdlLogList) #endif // USE_SDL2 } diff --git a/src/gui/widgets/tabs/setup_perfomance.cpp b/src/gui/widgets/tabs/setup_perfomance.cpp index 03a4fba45..83dc41503 100644 --- a/src/gui/widgets/tabs/setup_perfomance.cpp +++ b/src/gui/widgets/tabs/setup_perfomance.cpp @@ -229,8 +229,8 @@ Setup_Perfomance::Setup_Perfomance(const Widget2 *const widget) : Setup_Perfomance::~Setup_Perfomance() { - delete2(mTexturesList); + delete2(mTexturesList) #ifdef USE_SDL2 - delete2(mSdlDriversList); + delete2(mSdlDriversList) #endif // USE_SDL2 } diff --git a/src/gui/widgets/tabs/setup_players.cpp b/src/gui/widgets/tabs/setup_players.cpp index 67fe2d459..940c4d9c6 100644 --- a/src/gui/widgets/tabs/setup_players.cpp +++ b/src/gui/widgets/tabs/setup_players.cpp @@ -215,9 +215,9 @@ Setup_Players::Setup_Players(const Widget2 *const widget) : Setup_Players::~Setup_Players() { - delete2(mBadgesList); - delete2(mVisibleNamesList); - delete2(mVisibleNamesPosList); + delete2(mBadgesList) + delete2(mVisibleNamesList) + delete2(mVisibleNamesPosList) } void Setup_Players::externalUpdated() diff --git a/src/gui/widgets/tabs/setup_quick.cpp b/src/gui/widgets/tabs/setup_quick.cpp index f50c1d6a8..18e61d0c0 100644 --- a/src/gui/widgets/tabs/setup_quick.cpp +++ b/src/gui/widgets/tabs/setup_quick.cpp @@ -113,6 +113,6 @@ Setup_Quick::Setup_Quick(const Widget2 *const widget) : Setup_Quick::~Setup_Quick() { - delete2(mMoveTypeList); - delete2(mCrazyMoveTypeList); + delete2(mMoveTypeList) + delete2(mCrazyMoveTypeList) } diff --git a/src/gui/widgets/tabs/setup_relations.cpp b/src/gui/widgets/tabs/setup_relations.cpp index e89c2a3e0..03ee760c9 100644 --- a/src/gui/widgets/tabs/setup_relations.cpp +++ b/src/gui/widgets/tabs/setup_relations.cpp @@ -149,7 +149,7 @@ Setup_Relations::Setup_Relations(const Widget2 *const widget) : Setup_Relations::~Setup_Relations() { playerRelations.removeListener(this); - delete2(mIgnoreActionChoicesModel); + delete2(mIgnoreActionChoicesModel) } diff --git a/src/gui/widgets/tabs/setup_theme.cpp b/src/gui/widgets/tabs/setup_theme.cpp index 080573c3c..22db49b34 100644 --- a/src/gui/widgets/tabs/setup_theme.cpp +++ b/src/gui/widgets/tabs/setup_theme.cpp @@ -236,13 +236,13 @@ Setup_Theme::Setup_Theme(const Widget2 *const widget) : Setup_Theme::~Setup_Theme() { - delete2(mInfo); - delete2(mThemesModel); - delete2(mFontsModel); - delete2(mFontSizeListModel); - delete2(mNpcFontSizeListModel); - delete2(mLangListModel); - delete2(mInfo); + delete2(mInfo) + delete2(mThemesModel) + delete2(mFontsModel) + delete2(mFontSizeListModel) + delete2(mNpcFontSizeListModel) + delete2(mLangListModel) + delete2(mInfo) } void Setup_Theme::updateInfo() @@ -347,7 +347,7 @@ void Setup_Theme::cancel() } #define updateField(name1, name2) if (!mInfo->name1.empty()) \ - name2 = mInfo->name1; + name2 = mInfo->name1 void Setup_Theme::apply() { diff --git a/src/gui/widgets/tabs/setup_touch.cpp b/src/gui/widgets/tabs/setup_touch.cpp index f8247d15d..1b6ce3ed4 100644 --- a/src/gui/widgets/tabs/setup_touch.cpp +++ b/src/gui/widgets/tabs/setup_touch.cpp @@ -137,7 +137,7 @@ Setup_Touch::Setup_Touch(const Widget2 *const widget) : Setup_Touch::~Setup_Touch() { - delete2(mSizeList); - delete2(mFormatList); - delete2(mActionsList); + delete2(mSizeList) + delete2(mFormatList) + delete2(mActionsList) } diff --git a/src/gui/widgets/tabs/setup_video.cpp b/src/gui/widgets/tabs/setup_video.cpp index 557cdffce..0adc55ec6 100644 --- a/src/gui/widgets/tabs/setup_video.cpp +++ b/src/gui/widgets/tabs/setup_video.cpp @@ -217,10 +217,10 @@ Setup_Video::Setup_Video(const Widget2 *const widget) : Setup_Video::~Setup_Video() { - delete2(mModeListModel); - delete2(mModeList); - delete2(mOpenGLListModel); - delete2(mDialog); + delete2(mModeListModel) + delete2(mModeList) + delete2(mOpenGLListModel) + delete2(mDialog) } void Setup_Video::apply() diff --git a/src/gui/widgets/tabs/setup_visual.cpp b/src/gui/widgets/tabs/setup_visual.cpp index 6912ef560..325732467 100644 --- a/src/gui/widgets/tabs/setup_visual.cpp +++ b/src/gui/widgets/tabs/setup_visual.cpp @@ -256,12 +256,12 @@ Setup_Visual::Setup_Visual(const Widget2 *const widget) : Setup_Visual::~Setup_Visual() { - delete2(mSpeachList); - delete2(mAmbientFxList); - delete2(mParticleList); - delete2(mParticleTypeList); - delete2(mVSyncList); - delete2(mScaleList); + delete2(mSpeachList) + delete2(mAmbientFxList) + delete2(mParticleList) + delete2(mParticleTypeList) + delete2(mVSyncList) + delete2(mScaleList) } void Setup_Visual::apply() diff --git a/src/gui/widgets/tabs/setuptabscroll.cpp b/src/gui/widgets/tabs/setuptabscroll.cpp index 1b2096c1f..1f9cceaba 100644 --- a/src/gui/widgets/tabs/setuptabscroll.cpp +++ b/src/gui/widgets/tabs/setuptabscroll.cpp @@ -45,7 +45,7 @@ SetupTabScroll::SetupTabScroll(const Widget2 *const widget) : SetupTabScroll::~SetupTabScroll() { mScroll = nullptr; - delete2(mContainer); + delete2(mContainer) removeItems(); } diff --git a/src/gui/widgets/tabs/socialattacktab.h b/src/gui/widgets/tabs/socialattacktab.h index 0b2e8db41..752a32942 100644 --- a/src/gui/widgets/tabs/socialattacktab.h +++ b/src/gui/widgets/tabs/socialattacktab.h @@ -63,11 +63,11 @@ class SocialAttackTab final : public SocialTab { updateAtkListStart(); // TRANSLATORS: mobs group name in social window - addAvatars(PriorityAttackMob, _("Priority mobs"), PRIORITY); + addAvatars(PriorityAttackMob, _("Priority mobs"), PRIORITY) // TRANSLATORS: mobs group name in social window - addAvatars(AttackMob, _("Attack mobs"), ATTACK); + addAvatars(AttackMob, _("Attack mobs"), ATTACK) // TRANSLATORS: mobs group name in social window - addAvatars(IgnoreAttackMob, _("Ignore mobs"), IGNORE_); + addAvatars(IgnoreAttackMob, _("Ignore mobs"), IGNORE_) } private: diff --git a/src/gui/widgets/tabs/socialpickuptab.h b/src/gui/widgets/tabs/socialpickuptab.h index 54d860664..00dc616f1 100644 --- a/src/gui/widgets/tabs/socialpickuptab.h +++ b/src/gui/widgets/tabs/socialpickuptab.h @@ -63,9 +63,9 @@ class SocialPickupTab final : public SocialTab { updateAtkListStart(); // TRANSLATORS: items group name in social window - addAvatars(PickupItem, _("Pickup items"), PICKUP); + addAvatars(PickupItem, _("Pickup items"), PICKUP) // TRANSLATORS: items group name in social window - addAvatars(IgnorePickupItem, _("Ignore items"), NOPICKUP); + addAvatars(IgnorePickupItem, _("Ignore items"), NOPICKUP) } private: diff --git a/src/gui/widgets/tabs/socialtabbase.h b/src/gui/widgets/tabs/socialtabbase.h index b7cbf5a8f..8c0a7081e 100644 --- a/src/gui/widgets/tabs/socialtabbase.h +++ b/src/gui/widgets/tabs/socialtabbase.h @@ -79,6 +79,6 @@ ENUMS_RESOURCES_MAP_MAPITEMTYPE_H Avatar *ava = nullptr;\ std::list<std::string> mobs;\ std::list<std::string>::const_iterator i;\ - std::list<std::string>::const_iterator i_end; + std::list<std::string>::const_iterator i_end #endif // GUI_WIDGETS_TABS_SOCIALTABBASE_H diff --git a/src/gui/widgets/tabs/tab.cpp b/src/gui/widgets/tabs/tab.cpp index fd4faafa1..e43833fe8 100644 --- a/src/gui/widgets/tabs/tab.cpp +++ b/src/gui/widgets/tabs/tab.cpp @@ -140,14 +140,14 @@ Tab::~Tab() theme->unload(tabImg[mode]); } - delete2(mLabel); + delete2(mLabel) if (mImage != nullptr) { mImage->decRef(); mImage = nullptr; } - delete2(mVertexes); + delete2(mVertexes) } void Tab::init() diff --git a/src/gui/widgets/window.cpp b/src/gui/widgets/window.cpp index f76c5fb9e..be0712315 100644 --- a/src/gui/widgets/window.cpp +++ b/src/gui/widgets/window.cpp @@ -179,7 +179,7 @@ Window::Window(const std::string &caption, if (skin.empty()) { reportAlways("Default skin was used for window: %s", - caption.c_str()); + caption.c_str()) skin = "window.xml"; } @@ -251,7 +251,7 @@ void Window::postInit() { reportAlways("error: Window created with calling postInit() " "more than once: %s", - mWindowName.c_str()); + mWindowName.c_str()) } mInit = true; } @@ -272,7 +272,7 @@ Window::~Window() saveWindowState(); - delete2(mLayout); + delete2(mLayout) while (!mWidgets.empty()) delete mWidgets.front(); @@ -280,7 +280,7 @@ Window::~Window() mWidgets.clear(); removeWidgetListener(this); - delete2(mVertexes); + delete2(mVertexes) windowInstances--; @@ -298,7 +298,7 @@ Window::~Window() if (!mInit) { reportAlways("error: Window created without calling postInit(): %s", - mWindowName.c_str()); + mWindowName.c_str()) } } @@ -1398,7 +1398,7 @@ void Window::reflowLayout(int w, int h) return; mLayout->reflow(w, h); - delete2(mLayout); + delete2(mLayout) setContentSize(w, h); } diff --git a/src/gui/widgets/windowcontainer.cpp b/src/gui/widgets/windowcontainer.cpp index a275b7cb1..cb76223e3 100644 --- a/src/gui/widgets/windowcontainer.cpp +++ b/src/gui/widgets/windowcontainer.cpp @@ -59,7 +59,7 @@ void WindowContainer::scheduleDelete(Widget *const widget) else { reportAlways("double adding pointer %p for deletion in scheduleDelete", - static_cast<void*>(widget)); + static_cast<void*>(widget)) } } diff --git a/src/gui/windowmanager.cpp b/src/gui/windowmanager.cpp index d4923c478..b2adafff8 100644 --- a/src/gui/windowmanager.cpp +++ b/src/gui/windowmanager.cpp @@ -132,7 +132,7 @@ void WindowManager::createWindows() CREATEWIDGETV0(itemPopup, ItemPopup); CREATEWIDGETV0(spellPopup, SpellPopup); CREATEWIDGETV0(skillPopup, SkillPopup); - delete2(debugChatTab); + delete2(debugChatTab) if (chatWindow != nullptr) { chatWindow->scheduleDelete(); @@ -171,34 +171,34 @@ void WindowManager::createValidateWindows() void WindowManager::deleteValidateWindows() { #ifndef DYECMD - delete2(skillDialog); - delete2(beingEquipmentWindow); - delete2(questsWindow); + delete2(skillDialog) + delete2(beingEquipmentWindow) + delete2(questsWindow) #endif // DYECMD } void WindowManager::deleteWindows() { #ifndef DYECMD - delete2(textBoxPopup); - delete2(beingPopup); - delete2(itemPopup); - delete2(spellPopup); - delete2(skillPopup); - delete2(popupMenu); - delete2(didYouKnowWindow); - delete2(helpWindow); - delete2(setupWindow); - delete2(userPalette); + delete2(textBoxPopup) + delete2(beingPopup) + delete2(itemPopup) + delete2(spellPopup) + delete2(skillPopup) + delete2(popupMenu) + delete2(didYouKnowWindow) + delete2(helpWindow) + delete2(setupWindow) + delete2(userPalette) delete2(spellManager) delete2(spellShortcut) - delete2(debugChatTab); - delete2(chatWindow); - delete2(debugWindow); + delete2(debugChatTab) + delete2(chatWindow) + delete2(debugWindow) #endif // DYECMD - delete2(textPopup); + delete2(textPopup) } void WindowManager::initTitle() diff --git a/src/gui/windows/buydialog.cpp b/src/gui/windows/buydialog.cpp index cc4cf4699..0dbcac79c 100644 --- a/src/gui/windows/buydialog.cpp +++ b/src/gui/windows/buydialog.cpp @@ -426,8 +426,8 @@ void BuyDialog::init() BuyDialog::~BuyDialog() { - delete2(mShopItems); - delete2(mSortModel); + delete2(mShopItems) + delete2(mSortModel) instances.remove(this); if (buySellHandler != nullptr) buySellHandler->cleanDialogReference(this); diff --git a/src/gui/windows/changeemaildialog.cpp b/src/gui/windows/changeemaildialog.cpp index 5cd03a887..034ac2463 100644 --- a/src/gui/windows/changeemaildialog.cpp +++ b/src/gui/windows/changeemaildialog.cpp @@ -108,7 +108,7 @@ ChangeEmailDialog::ChangeEmailDialog(LoginData &data) : ChangeEmailDialog::~ChangeEmailDialog() { - delete2(mWrongDataNoticeListener); + delete2(mWrongDataNoticeListener) } void ChangeEmailDialog::action(const ActionEvent &event) diff --git a/src/gui/windows/changepassworddialog.cpp b/src/gui/windows/changepassworddialog.cpp index d302244ba..c0ba5e573 100644 --- a/src/gui/windows/changepassworddialog.cpp +++ b/src/gui/windows/changepassworddialog.cpp @@ -85,7 +85,7 @@ ChangePasswordDialog::ChangePasswordDialog(LoginData &data) : ChangePasswordDialog::~ChangePasswordDialog() { - delete2(mWrongDataNoticeListener); + delete2(mWrongDataNoticeListener) } void ChangePasswordDialog::action(const ActionEvent &event) diff --git a/src/gui/windows/charcreatedialog.cpp b/src/gui/windows/charcreatedialog.cpp index 852dbc2cc..5081ed63b 100644 --- a/src/gui/windows/charcreatedialog.cpp +++ b/src/gui/windows/charcreatedialog.cpp @@ -460,7 +460,7 @@ CharCreateDialog::CharCreateDialog(CharSelectDialog *const parent, CharCreateDialog::~CharCreateDialog() { - delete2(mPlayer); + delete2(mPlayer) if (charServerHandler != nullptr) charServerHandler->setCharCreateDialog(nullptr); diff --git a/src/gui/windows/chatwindow.cpp b/src/gui/windows/chatwindow.cpp index 560826d79..d981c49b9 100644 --- a/src/gui/windows/chatwindow.cpp +++ b/src/gui/windows/chatwindow.cpp @@ -219,9 +219,9 @@ ChatWindow::~ChatWindow() config.setValue("ReturnToggles", mReturnToggles); removeAllWhispers(); removeAllChannels(); - delete2(mItemLinkHandler); - delete2(mColorPicker); - delete2(mColorListModel); + delete2(mItemLinkHandler) + delete2(mColorPicker) + delete2(mColorListModel) } void ChatWindow::postInit() @@ -1060,7 +1060,7 @@ void ChatWindow::attributeChanged(const AttributesT id, } default: break; - }; + } PRAGMA45(GCC diagnostic pop) } diff --git a/src/gui/windows/clanwindow.cpp b/src/gui/windows/clanwindow.cpp index d2fb1bfd6..7520fdf7f 100644 --- a/src/gui/windows/clanwindow.cpp +++ b/src/gui/windows/clanwindow.cpp @@ -83,10 +83,10 @@ ClanWindow::ClanWindow() : ClanWindow::~ClanWindow() { - delete2(mInfoWidget); - delete2(mStatsWidget); - delete2(mAllyWidget); - delete2(mAntagonistWidget); + delete2(mInfoWidget) + delete2(mStatsWidget) + delete2(mAllyWidget) + delete2(mAntagonistWidget) } void ClanWindow::postInit() diff --git a/src/gui/windows/cutinwindow.cpp b/src/gui/windows/cutinwindow.cpp index 12a54a3b9..9b665b49e 100644 --- a/src/gui/windows/cutinwindow.cpp +++ b/src/gui/windows/cutinwindow.cpp @@ -52,7 +52,7 @@ CutInWindow::CutInWindow() : CutInWindow::~CutInWindow() { - delete2(mImage); + delete2(mImage) } void CutInWindow::draw(Graphics *const graphics) @@ -76,7 +76,7 @@ void CutInWindow::draw2(Graphics *const graphics) void CutInWindow::show(const std::string &name, const CutInT cutin) { - delete2(mImage); + delete2(mImage) if (name.empty()) { setVisible(Visible_false); @@ -138,7 +138,7 @@ void CutInWindow::show(const std::string &name, void CutInWindow::hide() { - delete2(mImage); + delete2(mImage) setVisible(Visible_false); } diff --git a/src/gui/windows/debugwindow.cpp b/src/gui/windows/debugwindow.cpp index 35d114ede..39cccba98 100644 --- a/src/gui/windows/debugwindow.cpp +++ b/src/gui/windows/debugwindow.cpp @@ -87,10 +87,10 @@ DebugWindow::DebugWindow(const std::string &name) : DebugWindow::~DebugWindow() { - delete2(mMapWidget); - delete2(mTargetWidget); - delete2(mNetWidget); - delete2(mStatWidget); + delete2(mMapWidget) + delete2(mTargetWidget) + delete2(mNetWidget) + delete2(mStatWidget) } void DebugWindow::postInit() diff --git a/src/gui/windows/didyouknowwindow.cpp b/src/gui/windows/didyouknowwindow.cpp index 1d53a9a9f..6f44b7c92 100644 --- a/src/gui/windows/didyouknowwindow.cpp +++ b/src/gui/windows/didyouknowwindow.cpp @@ -116,7 +116,7 @@ void DidYouKnowWindow::postInit() DidYouKnowWindow::~DidYouKnowWindow() { - delete2(mItemLinkHandler); + delete2(mItemLinkHandler) } void DidYouKnowWindow::action(const ActionEvent &event) diff --git a/src/gui/windows/emotewindow.cpp b/src/gui/windows/emotewindow.cpp index 537297203..17610da3c 100644 --- a/src/gui/windows/emotewindow.cpp +++ b/src/gui/windows/emotewindow.cpp @@ -146,16 +146,16 @@ EmoteWindow::~EmoteWindow() { mTabs->removeAll(false); mTabs->removeTab(mTabs->getTabByIndex(0)); - delete2(mEmotePage); - delete2(mColorPage); - delete2(mColorModel); - delete2(mScrollColorPage); - delete2(mFontPage); - delete2(mFontModel); - delete2(mScrollFontPage); - delete2(mTextPage); - delete2(mTextModel); - delete2(mScrollTextPage); + delete2(mEmotePage) + delete2(mColorPage) + delete2(mColorModel) + delete2(mScrollColorPage) + delete2(mFontPage) + delete2(mFontModel) + delete2(mScrollFontPage) + delete2(mTextPage) + delete2(mTextModel) + delete2(mScrollTextPage) if (mImageSet != nullptr) { mImageSet->decRef(); diff --git a/src/gui/windows/equipmentwindow.cpp b/src/gui/windows/equipmentwindow.cpp index 7d0284fc1..9a351e4da 100644 --- a/src/gui/windows/equipmentwindow.cpp +++ b/src/gui/windows/equipmentwindow.cpp @@ -175,7 +175,7 @@ EquipmentWindow::~EquipmentWindow() mSlotBackground->decRef(); if (mSlotHighlightedBackground != nullptr) mSlotHighlightedBackground->decRef(); - delete2(mVertexes); + delete2(mVertexes) } void EquipmentWindow::draw(Graphics *const graphics) @@ -751,7 +751,7 @@ void EquipmentWindow::prepareSlotNames() const std::string name = XML::getProperty(slotNode, "name", ""); if (name.empty()) { - reportAlways("Empty slot name detected."); + reportAlways("Empty slot name detected.") continue; } @@ -760,7 +760,7 @@ void EquipmentWindow::prepareSlotNames() { reportAlways("Wrong slot id '%d' for slot with name '%s'", slot, - name.c_str()); + name.c_str()) continue; } mSlotNames[name] = slot; diff --git a/src/gui/windows/inventorywindow.cpp b/src/gui/windows/inventorywindow.cpp index be036a558..aa4d90258 100644 --- a/src/gui/windows/inventorywindow.cpp +++ b/src/gui/windows/inventorywindow.cpp @@ -146,7 +146,7 @@ InventoryWindow::InventoryWindow(Inventory *const inventory) : mSortDropDown->setSelected(config.getIntValue( "cartSortOrder")); break; - }; + } } else { @@ -374,7 +374,7 @@ InventoryWindow::InventoryWindow(Inventory *const inventory) : case InventoryType::Craft: case InventoryType::TypeEnd: break; - }; + } Layout &layout = getLayout(); layout.setRowHeight(2, LayoutType::SET); @@ -420,7 +420,7 @@ InventoryWindow::~InventoryWindow() invInstances.front()->updateDropButton(); mSortDropDown->hideDrop(false); - delete2(mSortModel); + delete2(mSortModel) } void InventoryWindow::storeSortOrder() const @@ -449,7 +449,7 @@ void InventoryWindow::storeSortOrder() const config.setValue("cartSortOrder", mSortDropDown->getSelected()); break; - }; + } } } diff --git a/src/gui/windows/itemamountwindow.cpp b/src/gui/windows/itemamountwindow.cpp index c30a91780..60b8ea6e4 100644 --- a/src/gui/windows/itemamountwindow.cpp +++ b/src/gui/windows/itemamountwindow.cpp @@ -351,7 +351,7 @@ void ItemAmountWindow::postInit() ItemAmountWindow::~ItemAmountWindow() { - delete2(mItemsModal); + delete2(mItemsModal) } // Show ItemTooltip diff --git a/src/gui/windows/logindialog.cpp b/src/gui/windows/logindialog.cpp index 53a50d4d5..855ebbde9 100644 --- a/src/gui/windows/logindialog.cpp +++ b/src/gui/windows/logindialog.cpp @@ -224,8 +224,8 @@ LoginDialog::~LoginDialog() if (mUpdateHostDropDown != nullptr) mUpdateHostDropDown->hideDrop(false); - delete2(mUpdateTypeModel); - delete2(mUpdateListModel); + delete2(mUpdateTypeModel) + delete2(mUpdateListModel) } void LoginDialog::action(const ActionEvent &event) diff --git a/src/gui/windows/maileditwindow.cpp b/src/gui/windows/maileditwindow.cpp index c241d97f8..473e78ef4 100644 --- a/src/gui/windows/maileditwindow.cpp +++ b/src/gui/windows/maileditwindow.cpp @@ -133,7 +133,7 @@ MailEditWindow::MailEditWindow() : MailEditWindow::~MailEditWindow() { mailEditWindow = nullptr; - delete2(mInventory); + delete2(mInventory) } void MailEditWindow::action(const ActionEvent &event) diff --git a/src/gui/windows/mailviewwindow.cpp b/src/gui/windows/mailviewwindow.cpp index ea0aa9a91..61ed8faa2 100644 --- a/src/gui/windows/mailviewwindow.cpp +++ b/src/gui/windows/mailviewwindow.cpp @@ -152,10 +152,10 @@ MailViewWindow::~MailViewWindow() } else { - delete2(mMessage); - delete2(mGetMoneyButton); + delete2(mMessage) + delete2(mGetMoneyButton) } - delete2(mInventory); + delete2(mInventory) mailViewWindow = nullptr; } diff --git a/src/gui/windows/mailwindow.cpp b/src/gui/windows/mailwindow.cpp index 6240792b4..c0f55c1b1 100644 --- a/src/gui/windows/mailwindow.cpp +++ b/src/gui/windows/mailwindow.cpp @@ -118,9 +118,9 @@ MailWindow::MailWindow() : MailWindow::~MailWindow() { delete_all(mMessages); - delete2(mMailModel); - delete2(mailViewWindow); - delete2(mailEditWindow); + delete2(mMailModel) + delete2(mailViewWindow) + delete2(mailEditWindow) } void MailWindow::action(const ActionEvent &event) diff --git a/src/gui/windows/ministatuswindow.cpp b/src/gui/windows/ministatuswindow.cpp index 69dbcbf45..82facde3d 100644 --- a/src/gui/windows/ministatuswindow.cpp +++ b/src/gui/windows/ministatuswindow.cpp @@ -162,7 +162,7 @@ MiniStatusWindow::MiniStatusWindow() : MiniStatusWindow::~MiniStatusWindow() { - delete2(mStatusPopup); + delete2(mStatusPopup) delete_all(mIcons); mIcons.clear(); diff --git a/src/gui/windows/npcdialog.cpp b/src/gui/windows/npcdialog.cpp index 7a671c987..49f0f3919 100644 --- a/src/gui/windows/npcdialog.cpp +++ b/src/gui/windows/npcdialog.cpp @@ -256,25 +256,25 @@ NpcDialog::~NpcDialog() deleteSkinControls(); - delete2(mTextBox); - delete2(mClearButton); - delete2(mButton); - delete2(mButton2); - delete2(mButton3); - delete2(mScrollArea); - delete2(mItemList); - delete2(mTextField); - delete2(mIntField); - delete2(mResetButton); - delete2(mPlusButton); - delete2(mMinusButton); - delete2(mItemLinkHandler); - delete2(mItemContainer); - delete2(mInventory); - delete2(mComplexInventory); - delete2(mItemScrollArea); - delete2(mListScrollArea); - delete2(mSkinScrollArea); + delete2(mTextBox) + delete2(mClearButton) + delete2(mButton) + delete2(mButton2) + delete2(mButton3) + delete2(mScrollArea) + delete2(mItemList) + delete2(mTextField) + delete2(mIntField) + delete2(mResetButton) + delete2(mPlusButton) + delete2(mMinusButton) + delete2(mItemLinkHandler) + delete2(mItemContainer) + delete2(mInventory) + delete2(mComplexInventory) + delete2(mItemScrollArea) + delete2(mListScrollArea) + delete2(mSkinScrollArea) FOR_EACH (ImageVectorIter, it, mImages) { diff --git a/src/gui/windows/questswindow.cpp b/src/gui/windows/questswindow.cpp index 69243ab1d..9948cf03f 100644 --- a/src/gui/windows/questswindow.cpp +++ b/src/gui/windows/questswindow.cpp @@ -137,11 +137,11 @@ QuestsWindow::QuestsWindow() : QuestsWindow::~QuestsWindow() { - delete2(mQuestsModel); + delete2(mQuestsModel) QuestDb::unload(); - delete2(mItemLinkHandler); + delete2(mItemLinkHandler) mQuestLinks.clear(); mQuestReverseLinks.clear(); if (mCompleteIcon != nullptr) diff --git a/src/gui/windows/quitdialog.cpp b/src/gui/windows/quitdialog.cpp index 3484f9a48..2bddaa84b 100644 --- a/src/gui/windows/quitdialog.cpp +++ b/src/gui/windows/quitdialog.cpp @@ -133,10 +133,10 @@ QuitDialog::~QuitDialog() { if (mMyPointer != nullptr) *mMyPointer = nullptr; - delete2(mForceQuit); - delete2(mLogoutQuit); - delete2(mSwitchAccountServer); - delete2(mSwitchCharacter); + delete2(mForceQuit) + delete2(mLogoutQuit) + delete2(mSwitchAccountServer) + delete2(mSwitchCharacter) } void QuitDialog::placeOption(ContainerPlacer &placer, diff --git a/src/gui/windows/registerdialog.cpp b/src/gui/windows/registerdialog.cpp index 695fd58e2..5fb970f49 100644 --- a/src/gui/windows/registerdialog.cpp +++ b/src/gui/windows/registerdialog.cpp @@ -153,7 +153,7 @@ void RegisterDialog::postInit() RegisterDialog::~RegisterDialog() { - delete2(mWrongDataNoticeListener); + delete2(mWrongDataNoticeListener) } void RegisterDialog::action(const ActionEvent &event) diff --git a/src/gui/windows/serverdialog.cpp b/src/gui/windows/serverdialog.cpp index 9656d0c1c..f59f75ce2 100644 --- a/src/gui/windows/serverdialog.cpp +++ b/src/gui/windows/serverdialog.cpp @@ -219,7 +219,7 @@ ServerDialog::~ServerDialog() mDownload->cancel(); delete2(mDownload) } - delete2(mServersListModel); + delete2(mServersListModel) } void ServerDialog::connectToSelectedServer() diff --git a/src/gui/windows/setupwindow.cpp b/src/gui/windows/setupwindow.cpp index 46ac33da7..8b63a6a16 100644 --- a/src/gui/windows/setupwindow.cpp +++ b/src/gui/windows/setupwindow.cpp @@ -252,8 +252,8 @@ void SetupWindow::unloadAdditionalTabs() { unloadTab(mModsTab); unloadTab(mQuickTab); - delete2(mModsTab); - delete2(mQuickTab); + delete2(mModsTab) + delete2(mQuickTab) } void SetupWindow::externalUnload() diff --git a/src/gui/windows/shopwindow.cpp b/src/gui/windows/shopwindow.cpp index 91c7063de..e0306503d 100644 --- a/src/gui/windows/shopwindow.cpp +++ b/src/gui/windows/shopwindow.cpp @@ -248,11 +248,11 @@ ShopWindow::~ShopWindow() { saveList(); - delete2(mBuyShopItemList); - delete2(mSellShopItemList); - delete2(mBuyShopItems); - delete2(mSellShopItems); - delete2(mTradeItem); + delete2(mBuyShopItemList) + delete2(mSellShopItemList) + delete2(mBuyShopItems) + delete2(mSellShopItems) + delete2(mTradeItem) instances.remove(this); } @@ -560,7 +560,7 @@ void ShopWindow::loadList() if (!shopFile.is_open()) { reportAlways("Error opening file for reading: %s", - shopListName.c_str()); + shopListName.c_str()) shopFile.close(); return; } @@ -616,7 +616,7 @@ void ShopWindow::saveList() const if (!shopFile.is_open()) { reportAlways("Error opening file writing: %s", - shopListName.c_str()); + shopListName.c_str()) return; } diff --git a/src/gui/windows/shortcutwindow.cpp b/src/gui/windows/shortcutwindow.cpp index 6d431c37c..40ddf6930 100644 --- a/src/gui/windows/shortcutwindow.cpp +++ b/src/gui/windows/shortcutwindow.cpp @@ -154,8 +154,8 @@ ShortcutWindow::~ShortcutWindow() { if (mTabs != nullptr) mTabs->removeAll(true); - delete2(mTabs); - delete2(mItems); + delete2(mTabs) + delete2(mItems) } void ShortcutWindow::addButton(const std::string &text, diff --git a/src/gui/windows/skilldialog.cpp b/src/gui/windows/skilldialog.cpp index 39235c570..2dd536c29 100644 --- a/src/gui/windows/skilldialog.cpp +++ b/src/gui/windows/skilldialog.cpp @@ -360,7 +360,7 @@ void SkillDialog::loadSkills() if (mSkills.empty()) loadXmlFile(paths.getStringValue("skillsFile2"), SkipError_false); loadXmlFile(paths.getStringValue("skillsPatchFile"), SkipError_true); - loadXmlDir("skillsPatchDir", loadXmlFile); + loadXmlDir("skillsPatchDir", loadXmlFile) addDefaultTab(); update(); @@ -452,7 +452,7 @@ void SkillDialog::loadXmlFile(const std::string &fileName, } default: reportAlways("Unsupported skillset type: %s", - setTypeStr.c_str()); + setTypeStr.c_str()) return; } if (mDefaultModel == nullptr) @@ -1191,7 +1191,7 @@ void SkillDialog::useSkillTarget(const SkillInfo *const info, } else { - reportAlways("Unsupported skill type: %d", type); + reportAlways("Unsupported skill type: %d", type) } } @@ -1276,7 +1276,7 @@ void SkillDialog::useSkillPosition(const SkillInfo *const info, } else { - reportAlways("Unsupported skill type: %d", type); + reportAlways("Unsupported skill type: %d", type) } } @@ -1378,7 +1378,7 @@ void SkillDialog::useSkillDefault(const SkillInfo *const info, } else { - reportAlways("Unsupported skill type: %d", type); + reportAlways("Unsupported skill type: %d", type) } } diff --git a/src/gui/windows/socialwindow.cpp b/src/gui/windows/socialwindow.cpp index ab469cada..8cf7419a1 100644 --- a/src/gui/windows/socialwindow.cpp +++ b/src/gui/windows/socialwindow.cpp @@ -166,11 +166,11 @@ SocialWindow::~SocialWindow() mPartyInviter.clear(); } - delete2(mPlayers); - delete2(mNavigation); - delete2(mAttackFilter); - delete2(mPickupFilter); - delete2(mFriends); + delete2(mPlayers) + delete2(mNavigation) + delete2(mAttackFilter) + delete2(mPickupFilter) + delete2(mFriends) FOR_EACH (GuildMap::iterator, it, mGuilds) { delete (*it).second; diff --git a/src/gui/windows/statuswindow.cpp b/src/gui/windows/statuswindow.cpp index 3403e1b4c..4795ddb53 100644 --- a/src/gui/windows/statuswindow.cpp +++ b/src/gui/windows/statuswindow.cpp @@ -231,7 +231,7 @@ StatusWindow::StatusWindow() : StatusWindow::~StatusWindow() { - delete2(mBasicStatsPage); + delete2(mBasicStatsPage) delete_all(mPages); } diff --git a/src/gui/windows/textcommandeditor.cpp b/src/gui/windows/textcommandeditor.cpp index c9d48f21c..f22551864 100644 --- a/src/gui/windows/textcommandeditor.cpp +++ b/src/gui/windows/textcommandeditor.cpp @@ -245,10 +245,10 @@ void TextCommandEditor::postInit() TextCommandEditor::~TextCommandEditor() { - delete2(mIconsModel); - delete2(mTargetTypeModel); + delete2(mIconsModel) + delete2(mTargetTypeModel) #ifdef TMWA_SUPPORT - delete2(mMagicSchoolModel); + delete2(mMagicSchoolModel) #endif // TMWA_SUPPORT } diff --git a/src/gui/windows/textselectdialog.cpp b/src/gui/windows/textselectdialog.cpp index 9991a5754..7049c478b 100644 --- a/src/gui/windows/textselectdialog.cpp +++ b/src/gui/windows/textselectdialog.cpp @@ -132,7 +132,7 @@ void TextSelectDialog::postInit() TextSelectDialog::~TextSelectDialog() { - delete2(mModel); + delete2(mModel) } void TextSelectDialog::action(const ActionEvent &event) diff --git a/src/gui/windows/tradewindow.cpp b/src/gui/windows/tradewindow.cpp index 606d2a380..8e7842a69 100644 --- a/src/gui/windows/tradewindow.cpp +++ b/src/gui/windows/tradewindow.cpp @@ -166,8 +166,8 @@ TradeWindow::TradeWindow() : TradeWindow::~TradeWindow() { - delete2(mMyInventory); - delete2(mPartnerInventory); + delete2(mMyInventory) + delete2(mPartnerInventory) } void TradeWindow::setMoney(const int amount) diff --git a/src/gui/windows/updaterwindow.cpp b/src/gui/windows/updaterwindow.cpp index 5b810b38e..f8951b308 100644 --- a/src/gui/windows/updaterwindow.cpp +++ b/src/gui/windows/updaterwindow.cpp @@ -275,7 +275,7 @@ UpdaterWindow::~UpdaterWindow() delete2(mDownload) } free(mMemoryBuffer); - delete2(mItemLinkHandler); + delete2(mItemLinkHandler) } void UpdaterWindow::setProgress(const float p) diff --git a/src/gui/windows/worldselectdialog.cpp b/src/gui/windows/worldselectdialog.cpp index 86e6a0e87..ed40b6269 100644 --- a/src/gui/windows/worldselectdialog.cpp +++ b/src/gui/windows/worldselectdialog.cpp @@ -96,7 +96,7 @@ void WorldSelectDialog::postInit() WorldSelectDialog::~WorldSelectDialog() { - delete2(mWorldListModel); + delete2(mWorldListModel) } void WorldSelectDialog::action(const ActionEvent &event) diff --git a/src/input/inputmanager.cpp b/src/input/inputmanager.cpp index a9a0fc29f..3e32621a1 100644 --- a/src/input/inputmanager.cpp +++ b/src/input/inputmanager.cpp @@ -906,7 +906,7 @@ bool InputManager::executeChatCommand(const std::string &restrict cmd, { reportAlways("Unknown chat command: /%s %s", cmd.c_str(), - args.c_str()); + args.c_str()) } return false; } diff --git a/src/input/joystick.cpp b/src/input/joystick.cpp index 91e21dc1c..4f7c6e9aa 100644 --- a/src/input/joystick.cpp +++ b/src/input/joystick.cpp @@ -227,7 +227,7 @@ void Joystick::logic() doCalibration(); BLOCK_END("Joystick::logic") return; - }; + } if (!mEnabled || !mCalibrated) { diff --git a/src/input/touch/touchmanager.cpp b/src/input/touch/touchmanager.cpp index 95fcd26f6..8195a4640 100644 --- a/src/input/touch/touchmanager.cpp +++ b/src/input/touch/touchmanager.cpp @@ -200,7 +200,7 @@ void TouchManager::clear() restrict2 FOR_EACH (TouchItemVectorCIter, it, mObjects) unload(*it); mObjects.clear(); - delete2(mVertexes); + delete2(mVertexes) mRedraw = true; } @@ -394,7 +394,7 @@ void TouchManager::unload(TouchItem *restrict const item) if (item->images != nullptr) { Theme::unloadRect(*item->images, 0, 8); - delete2(item->images); + delete2(item->images) if (item->icon != nullptr) { item->icon->decRef(); @@ -647,7 +647,7 @@ void TouchManager::loadButtons() restrict2 nullptr, nullptr, nullptr, nullptr); break; } - }; + } theme->unload(skin); } } diff --git a/src/maingui.cpp b/src/maingui.cpp index 908e2392b..e8f1cea12 100644 --- a/src/maingui.cpp +++ b/src/maingui.cpp @@ -149,7 +149,7 @@ int mainGui(int argc, char *argv[]) client->testsInit(); ret = Client::testsExec(); } - delete2(client); + delete2(client) VirtFs::deinit(); #if SDL_MIXER_VERSION_ATLEAST(1, 2, 11) diff --git a/src/net/character.h b/src/net/character.h index 3041c37a6..3c0c8f534 100644 --- a/src/net/character.h +++ b/src/net/character.h @@ -49,7 +49,7 @@ struct Character final ~Character() { - delete2(dummy); + delete2(dummy) } LocalPlayer *dummy; /**< A dummy representing this character */ diff --git a/src/net/ea/inventoryhandler.cpp b/src/net/ea/inventoryhandler.cpp index faed9df20..848b8db91 100644 --- a/src/net/ea/inventoryhandler.cpp +++ b/src/net/ea/inventoryhandler.cpp @@ -52,12 +52,12 @@ InventoryHandler::~InventoryHandler() storageWindow = nullptr; } - delete2(InventoryRecv::mStorage); + delete2(InventoryRecv::mStorage) } void InventoryHandler::clear() const { - delete2(InventoryRecv::mStorage); + delete2(InventoryRecv::mStorage) } bool InventoryHandler::canSplit(const Item *const item A_UNUSED) const diff --git a/src/net/ea/inventoryrecv.cpp b/src/net/ea/inventoryrecv.cpp index 2e6a7e2ac..f95d0d40d 100644 --- a/src/net/ea/inventoryrecv.cpp +++ b/src/net/ea/inventoryrecv.cpp @@ -137,7 +137,7 @@ void InventoryRecv::processPlayerStorageClose(Net::MessageIn &msg A_UNUSED) if (mStorage != nullptr) mStorage->clear(); - delete2(mStorage); + delete2(mStorage) BLOCK_END("InventoryRecv::processPlayerStorageClose") } diff --git a/src/net/ea/network.cpp b/src/net/ea/network.cpp index 11a0757f1..4a4c3d0df 100644 --- a/src/net/ea/network.cpp +++ b/src/net/ea/network.cpp @@ -90,9 +90,9 @@ Network::~Network() SDL_DestroyMutex(mMutexOut); mMutexOut = nullptr; - delete2Arr(mInBuffer); - delete2Arr(mOutBuffer); - delete2Arr(mPackets); + delete2Arr(mInBuffer) + delete2Arr(mOutBuffer) + delete2Arr(mPackets) TcpNet::quit(); } diff --git a/src/net/ea/partyhandler.cpp b/src/net/ea/partyhandler.cpp index 715073a46..651d29be5 100644 --- a/src/net/ea/partyhandler.cpp +++ b/src/net/ea/partyhandler.cpp @@ -44,7 +44,7 @@ PartyHandler::PartyHandler() : PartyHandler::~PartyHandler() { - delete2(partyTab); + delete2(partyTab) taParty = nullptr; } diff --git a/src/net/eathena/beingrecv.cpp b/src/net/eathena/beingrecv.cpp index ec7f4dc09..dc6a503bf 100644 --- a/src/net/eathena/beingrecv.cpp +++ b/src/net/eathena/beingrecv.cpp @@ -189,7 +189,7 @@ static void setBasicFields(Being *restrict const dstBeing, case ActorType::Avatar: case ActorType::Unknown: reportAlways("Wrong being type detected: %d", - CAST_S32(actorType)); + CAST_S32(actorType)) break; } } diff --git a/src/net/eathena/buysellrecv.cpp b/src/net/eathena/buysellrecv.cpp index 9bb91520b..7f71a41ab 100644 --- a/src/net/eathena/buysellrecv.cpp +++ b/src/net/eathena/buysellrecv.cpp @@ -137,7 +137,7 @@ void BuySellRecv::processNpcBuyResponse(Net::MessageIn &msg) default: NotifyManager::notify(NotifyTypes::BUY_FAILED); break; - }; + } } } // namespace EAthena diff --git a/src/net/eathena/clanrecv.cpp b/src/net/eathena/clanrecv.cpp index c2c5680db..e84ac7d3d 100644 --- a/src/net/eathena/clanrecv.cpp +++ b/src/net/eathena/clanrecv.cpp @@ -66,7 +66,7 @@ void ClanRecv::processClanInfo(Net::MessageIn &msg) if (info == nullptr) { reportAlways("missing clan %d in clandb", - localClan.id); + localClan.id) } else { @@ -85,7 +85,7 @@ void ClanRecv::processClanOnlineCount(Net::MessageIn &msg) void ClanRecv::processClanLeave(Net::MessageIn &msg A_UNUSED) { - delete2(clanTab); + delete2(clanTab) localClan.clear(); clanWindow->resetClan(); } @@ -99,7 +99,7 @@ void ClanRecv::processClanChat(Net::MessageIn &msg) std::string chatMsg = msg.readString(chatMsgLength, "message"); if (clanTab == nullptr) { - reportAlways("received clan chat messages without clan."); + reportAlways("received clan chat messages without clan.") return; } const size_t pos = chatMsg.find(" : ", 0); diff --git a/src/net/eathena/elementalrecv.cpp b/src/net/eathena/elementalrecv.cpp index 9d97b10b7..071a74c79 100644 --- a/src/net/eathena/elementalrecv.cpp +++ b/src/net/eathena/elementalrecv.cpp @@ -38,7 +38,7 @@ namespace EAthena PlayerInfo::setStatBase(stat, \ val, \ Notify_true); \ - break; + break void ElementalRecv::processElementalUpdateStatus(Net::MessageIn &msg) { @@ -52,7 +52,7 @@ void ElementalRecv::processElementalUpdateStatus(Net::MessageIn &msg) setElementalStat(Sp::MAXSP, Attributes::ELEMENTAL_MAX_MP); default: reportAlways("Unknown elemental stat %d", - sp); + sp) break; } } diff --git a/src/net/eathena/generalhandler.cpp b/src/net/eathena/generalhandler.cpp index 940a27fd8..8c1063b84 100644 --- a/src/net/eathena/generalhandler.cpp +++ b/src/net/eathena/generalhandler.cpp @@ -126,46 +126,46 @@ GeneralHandler::GeneralHandler() : GeneralHandler::~GeneralHandler() { - delete2(Network::mInstance); + delete2(Network::mInstance) - delete2(mAdminHandler); - delete2(mBeingHandler); - delete2(mBuySellHandler); - delete2(mCharServerHandler); - delete2(mChatHandler); - delete2(mClanHandler); - delete2(mGameHandler); - delete2(mGuildHandler); - delete2(mInventoryHandler); - delete2(mItemHandler); - delete2(mLoginHandler); - delete2(mNpcHandler); - delete2(mPartyHandler); - delete2(mPetHandler); - delete2(mPlayerHandler); - delete2(mSkillHandler); - delete2(mTradeHandler); - delete2(mQuestHandler); - delete2(mServerFeatures); - delete2(mMail2Handler); - delete2(mMailHandler); - delete2(mAuctionHandler); - delete2(mAchievementHandler); - delete2(mAttendanceHandler); - delete2(mCashShopHandler); - delete2(mFamilyHandler); - delete2(mBankHandler); - delete2(mBattleGroundHandler); - delete2(mMercenaryHandler); - delete2(mBuyingStoreHandler); - delete2(mHomunculusHandler); - delete2(mFriendsHandler); - delete2(mElementalHandler); - delete2(mMapHandler); - delete2(mMarketHandler); - delete2(mVendingHandler); - delete2(mRouletteHandler); - delete2(mSearchStoreHandler); + delete2(mAdminHandler) + delete2(mBeingHandler) + delete2(mBuySellHandler) + delete2(mCharServerHandler) + delete2(mChatHandler) + delete2(mClanHandler) + delete2(mGameHandler) + delete2(mGuildHandler) + delete2(mInventoryHandler) + delete2(mItemHandler) + delete2(mLoginHandler) + delete2(mNpcHandler) + delete2(mPartyHandler) + delete2(mPetHandler) + delete2(mPlayerHandler) + delete2(mSkillHandler) + delete2(mTradeHandler) + delete2(mQuestHandler) + delete2(mServerFeatures) + delete2(mMail2Handler) + delete2(mMailHandler) + delete2(mAuctionHandler) + delete2(mAchievementHandler) + delete2(mAttendanceHandler) + delete2(mCashShopHandler) + delete2(mFamilyHandler) + delete2(mBankHandler) + delete2(mBattleGroundHandler) + delete2(mMercenaryHandler) + delete2(mBuyingStoreHandler) + delete2(mHomunculusHandler) + delete2(mFriendsHandler) + delete2(mElementalHandler) + delete2(mMapHandler) + delete2(mMarketHandler) + delete2(mVendingHandler) + delete2(mRouletteHandler) + delete2(mSearchStoreHandler) } void GeneralHandler::load() const @@ -249,9 +249,9 @@ void GeneralHandler::gameEnded() const socialWindow->removeTab(Ea::taParty); } - delete2(guildTab); - delete2(partyTab); - delete2(clanTab); + delete2(guildTab) + delete2(partyTab) + delete2(clanTab) } } // namespace EAthena diff --git a/src/net/eathena/guildhandler.cpp b/src/net/eathena/guildhandler.cpp index 3cd3c2fba..320894d22 100644 --- a/src/net/eathena/guildhandler.cpp +++ b/src/net/eathena/guildhandler.cpp @@ -52,7 +52,7 @@ GuildHandler::GuildHandler() : GuildHandler::~GuildHandler() { - delete2(guildTab); + delete2(guildTab) guildHandler = nullptr; taGuild = nullptr; } diff --git a/src/net/eathena/guildrecv.cpp b/src/net/eathena/guildrecv.cpp index b5450e1aa..81010a230 100644 --- a/src/net/eathena/guildrecv.cpp +++ b/src/net/eathena/guildrecv.cpp @@ -208,7 +208,7 @@ void GuildRecv::processGuildMemberList(Net::MessageIn &msg) if (msg.getVersion() >= 20161026) { guildSize = 34; - reportAlways("missing guild member names"); + reportAlways("missing guild member names") } else { diff --git a/src/net/eathena/mail2handler.cpp b/src/net/eathena/mail2handler.cpp index 7097333bc..3a0c99ac0 100644 --- a/src/net/eathena/mail2handler.cpp +++ b/src/net/eathena/mail2handler.cpp @@ -108,7 +108,7 @@ void Mail2Handler::sendMail(const std::string &to, int32_t sz = 2 + 2 + 24 + 24 + 8 + 2 + 2 + titleSz + bodySz; if (sz > 32767 - 4) { - reportAlways("Mail message too big"); + reportAlways("Mail message too big") return; } if (packetVersion >= 20160600) diff --git a/src/net/eathena/mail2recv.cpp b/src/net/eathena/mail2recv.cpp index d22bc211e..2cb585829 100644 --- a/src/net/eathena/mail2recv.cpp +++ b/src/net/eathena/mail2recv.cpp @@ -104,14 +104,14 @@ void Mail2Recv::processAddItemResult(Net::MessageIn &msg) if (mailEditWindow == nullptr) { - reportAlways("Mail edit window not created"); + reportAlways("Mail edit window not created") delete options; return; } Inventory *const inventory = mailEditWindow->getInventory(); if (inventory == nullptr) { - reportAlways("Mail edit window inventory not exists"); + reportAlways("Mail edit window inventory not exists") delete options; return; } @@ -189,7 +189,7 @@ void Mail2Recv::processRemoveItemResult(Net::MessageIn &msg) const Inventory *const inv = PlayerInfo::getInventory(); if (inv == nullptr) { - reportAlways("Player inventory not exists"); + reportAlways("Player inventory not exists") return; } std::string itemName; @@ -211,25 +211,25 @@ void Mail2Recv::processRemoveItemResult(Net::MessageIn &msg) } if (mailEditWindow == nullptr) { - reportAlways("Mail edit window not created"); + reportAlways("Mail edit window not created") return; } Inventory *const inventory = mailEditWindow->getInventory(); if (inventory == nullptr) { - reportAlways("Mail edit window inventory not exists"); + reportAlways("Mail edit window inventory not exists") return; } const int index2 = inventory->findIndexByTag(index); if (index2 == -1) { - reportAlways("Item not exists in mail edit window."); + reportAlways("Item not exists in mail edit window.") return; } Item *const item = inventory->getItem(index2); if (item == nullptr) { - reportAlways("Item not exists."); + reportAlways("Item not exists.") return; } @@ -248,7 +248,7 @@ void Mail2Recv::processCheckNameResult(Net::MessageIn &msg) if (mMailQueue.empty()) { reportAlways("Mail2Recv::processCheckNameResult no names in queue." - "Char id: %d", charId); + "Char id: %d", charId) return; } MailQueue *const mail = mMailQueue.front(); @@ -272,7 +272,7 @@ void Mail2Recv::processCheckNameResult(Net::MessageIn &msg) case MailQueueType::EditMail: if (mailWindow == nullptr) { - reportAlways("Mail window not created"); + reportAlways("Mail window not created") } else { @@ -282,7 +282,7 @@ void Mail2Recv::processCheckNameResult(Net::MessageIn &msg) case MailQueueType::ValidateTo: if (mailEditWindow == nullptr) { - reportAlways("Mail edit window not created"); + reportAlways("Mail edit window not created") } else { @@ -291,7 +291,7 @@ void Mail2Recv::processCheckNameResult(Net::MessageIn &msg) break; case MailQueueType::Unknown: default: - reportAlways("Not implemented yet."); + reportAlways("Not implemented yet.") break; } delete mail; @@ -329,7 +329,7 @@ void Mail2Recv::processMailListPage(Net::MessageIn &msg) { if (mailWindow == nullptr) { - reportAlways("mail window not created"); + reportAlways("mail window not created") return; } msg.readInt16("len"); @@ -405,7 +405,7 @@ void Mail2Recv::processReadMail(Net::MessageIn &msg) if (mail == nullptr) { - reportAlways("Mail message not found"); + reportAlways("Mail message not found") for (int f = 0; f < itemsCount; f ++) { msg.readInt16("amount"); @@ -490,7 +490,7 @@ void Mail2Recv::processMailDelete(Net::MessageIn &msg) const int64_t mailId = msg.readInt64("mail id"); if (mailWindow == nullptr) { - reportAlways("Mail window not created."); + reportAlways("Mail window not created.") return; } mailWindow->removeMail(mailId); diff --git a/src/net/eathena/mailrecv.cpp b/src/net/eathena/mailrecv.cpp index 94e6c6fb9..d759e0fa3 100644 --- a/src/net/eathena/mailrecv.cpp +++ b/src/net/eathena/mailrecv.cpp @@ -122,21 +122,21 @@ void MailRecv::processReadMail(Net::MessageIn &msg) mail->strTime = timeToStr(mail->time); if (mailWindow == nullptr) { - reportAlways("Mail window not created"); + reportAlways("Mail window not created") delete mail; return; } mailWindow->showMessage(mail, itemId != 0 ? 1 : 0); if (mailViewWindow == nullptr) { - reportAlways("Mail view window not created"); + reportAlways("Mail view window not created") return; } Inventory *const inventory = mailViewWindow->getInventory(); if (inventory == nullptr) { - reportAlways("Mail view window missing inventory"); + reportAlways("Mail view window missing inventory") return; } diff --git a/src/net/eathena/mercenaryrecv.cpp b/src/net/eathena/mercenaryrecv.cpp index bd4579c83..e9d351b37 100644 --- a/src/net/eathena/mercenaryrecv.cpp +++ b/src/net/eathena/mercenaryrecv.cpp @@ -47,7 +47,7 @@ namespace EAthena PlayerInfo::setStatBase(stat, \ val, \ Notify_true); \ - break; + break void MercenaryRecv::processMercenaryUpdate(Net::MessageIn &msg) { @@ -71,7 +71,7 @@ void MercenaryRecv::processMercenaryUpdate(Net::MessageIn &msg) setMercStat(Sp::MERCFAITH, Attributes::MERC_FAITH); default: reportAlways("Unknown mercenary stat %d", - sp); + sp) break; } } diff --git a/src/net/eathena/network.cpp b/src/net/eathena/network.cpp index c183d842c..9fcccb378 100644 --- a/src/net/eathena/network.cpp +++ b/src/net/eathena/network.cpp @@ -133,7 +133,7 @@ void Network::registerFakeHandlers() const size_t id = (*it).first; if (id >= packet_lengths_size) { - reportAlways("Wrong fake packet id %d", CAST_S32(id)); + reportAlways("Wrong fake packet id %d", CAST_S32(id)) continue; } if (mPackets[id].len != 0 || @@ -159,7 +159,7 @@ void Network::registerFakeHandlers() const size_t id = *it; if (id >= packet_lengths_size) { - reportAlways("Wrong remove packet id %d", CAST_S32(id)); + reportAlways("Wrong remove packet id %d", CAST_S32(id)) continue; } if (mPackets[id].len == 0 && diff --git a/src/net/eathena/partyrecv.cpp b/src/net/eathena/partyrecv.cpp index 21b97dfad..026dafbfc 100644 --- a/src/net/eathena/partyrecv.cpp +++ b/src/net/eathena/partyrecv.cpp @@ -117,7 +117,7 @@ void PartyRecv::processPartyMemberJobLevel(Net::MessageIn &msg) } else { - reportAlways("processPartyMemberJobLevel: party member not exists."); + reportAlways("processPartyMemberJobLevel: party member not exists.") } } diff --git a/src/net/eathena/playerrecv.cpp b/src/net/eathena/playerrecv.cpp index 32336249d..da3e35a0d 100644 --- a/src/net/eathena/playerrecv.cpp +++ b/src/net/eathena/playerrecv.cpp @@ -507,7 +507,7 @@ void PlayerRecv::processKilledBy(Net::MessageIn &msg) { debugMsg( // TRANSLATORS: player killed message - _("You were killed by unknown source.")); + _("You were killed by unknown source.")) } else { @@ -519,7 +519,7 @@ void PlayerRecv::processKilledBy(Net::MessageIn &msg) debugMsg(strprintf( // TRANSLATORS: player killed message _("You were killed by %s."), - name.c_str())); + name.c_str())) } } diff --git a/src/net/ipc.cpp b/src/net/ipc.cpp index 2b4573cba..324ad4565 100644 --- a/src/net/ipc.cpp +++ b/src/net/ipc.cpp @@ -156,7 +156,7 @@ void IPC::stop() return; logger->log("Stopping IPC..."); - delete2(ipc); + delete2(ipc) } void IPC::start() @@ -183,7 +183,7 @@ void IPC::start() port ++; } - delete2(ipc); + delete2(ipc) } void IPC::flush() diff --git a/src/net/net.cpp b/src/net/net.cpp index f9422c367..98e81576c 100644 --- a/src/net/net.cpp +++ b/src/net/net.cpp @@ -136,7 +136,7 @@ void connectToServer(const ServerInfo &server) generalHandler != nullptr) { generalHandler->unload(); - delete2(generalHandler); + delete2(generalHandler) } switch (server.type) @@ -178,7 +178,7 @@ void unload() if (handler != nullptr) handler->unload(); ignorePackets.clear(); - delete2(generalHandler); + delete2(generalHandler) } ServerTypeT getNetworkType() diff --git a/src/net/packetlimiter.cpp b/src/net/packetlimiter.cpp index 50279b22c..dbf29ffd7 100644 --- a/src/net/packetlimiter.cpp +++ b/src/net/packetlimiter.cpp @@ -230,7 +230,7 @@ void PacketLimiter::writePacketLimits(const std::string &packetLimitsName) if (!outPacketFile.is_open()) { reportAlways("Error opening file for writing: %s", - packetLimitsName.c_str()); + packetLimitsName.c_str()) outPacketFile.close(); return; } diff --git a/src/net/tmwa/buysellrecv.cpp b/src/net/tmwa/buysellrecv.cpp index b30ebc501..17568c756 100644 --- a/src/net/tmwa/buysellrecv.cpp +++ b/src/net/tmwa/buysellrecv.cpp @@ -117,7 +117,7 @@ void BuySellRecv::processNpcBuyResponse(Net::MessageIn &msg) default: NotifyManager::notify(NotifyTypes::BUY_FAILED); break; - }; + } } } // namespace TmwAthena diff --git a/src/net/tmwa/generalhandler.cpp b/src/net/tmwa/generalhandler.cpp index 99865f61d..57509a717 100644 --- a/src/net/tmwa/generalhandler.cpp +++ b/src/net/tmwa/generalhandler.cpp @@ -125,46 +125,46 @@ GeneralHandler::GeneralHandler() : GeneralHandler::~GeneralHandler() { - delete2(Network::mInstance); - - delete2(mAdminHandler); - delete2(mBeingHandler); - delete2(mBuySellHandler); - delete2(mCharServerHandler); - delete2(mChatHandler); - delete2(mClanHandler); - delete2(mGameHandler); - delete2(mGuildHandler); - delete2(mInventoryHandler); - delete2(mItemHandler); - delete2(mLoginHandler); - delete2(mNpcHandler); - delete2(mPartyHandler); - delete2(mPetHandler); - delete2(mPlayerHandler); - delete2(mSkillHandler); - delete2(mTradeHandler); - delete2(mQuestHandler); - delete2(mServerFeatures); - delete2(mAuctionHandler); - delete2(mAchievementHandler); - delete2(mAttendanceHandler); - delete2(mBankHandler); - delete2(mBattleGroundHandler); - delete2(mBuyingStoreHandler); - delete2(mCashShopHandler); - delete2(mElementalHandler); - delete2(mFamilyHandler); - delete2(mFriendsHandler); - delete2(mHomunculusHandler); - delete2(mMail2Handler); - delete2(mMailHandler); - delete2(mMapHandler); - delete2(mMarketHandler); - delete2(mMercenaryHandler); - delete2(mRouletteHandler); - delete2(mSearchStoreHandler); - delete2(mVendingHandler); + delete2(Network::mInstance) + + delete2(mAdminHandler) + delete2(mBeingHandler) + delete2(mBuySellHandler) + delete2(mCharServerHandler) + delete2(mChatHandler) + delete2(mClanHandler) + delete2(mGameHandler) + delete2(mGuildHandler) + delete2(mInventoryHandler) + delete2(mItemHandler) + delete2(mLoginHandler) + delete2(mNpcHandler) + delete2(mPartyHandler) + delete2(mPetHandler) + delete2(mPlayerHandler) + delete2(mSkillHandler) + delete2(mTradeHandler) + delete2(mQuestHandler) + delete2(mServerFeatures) + delete2(mAuctionHandler) + delete2(mAchievementHandler) + delete2(mAttendanceHandler) + delete2(mBankHandler) + delete2(mBattleGroundHandler) + delete2(mBuyingStoreHandler) + delete2(mCashShopHandler) + delete2(mElementalHandler) + delete2(mFamilyHandler) + delete2(mFriendsHandler) + delete2(mHomunculusHandler) + delete2(mMail2Handler) + delete2(mMailHandler) + delete2(mMapHandler) + delete2(mMarketHandler) + delete2(mMercenaryHandler) + delete2(mRouletteHandler) + delete2(mSearchStoreHandler) + delete2(mVendingHandler) } void GeneralHandler::load() const @@ -249,7 +249,7 @@ void GeneralHandler::gameEnded() const if (socialWindow != nullptr) socialWindow->removeTab(Ea::taParty); - delete2(partyTab); + delete2(partyTab) } } // namespace TmwAthena diff --git a/src/net/tmwa/guildmanager.cpp b/src/net/tmwa/guildmanager.cpp index b491b250d..aaf8562ac 100644 --- a/src/net/tmwa/guildmanager.cpp +++ b/src/net/tmwa/guildmanager.cpp @@ -66,7 +66,7 @@ GuildManager::GuildManager() : GuildManager::~GuildManager() { - delete2(mTab); + delete2(mTab) } void GuildManager::init() @@ -90,7 +90,7 @@ void GuildManager::init() } else if (guildManager != nullptr) { - delete2(guildManager); + delete2(guildManager) } } @@ -112,7 +112,7 @@ void GuildManager::reload() if ((guild != nullptr) && (socialWindow != nullptr)) socialWindow->removeTab(guild); } - delete2(mTab); + delete2(mTab) } void GuildManager::send(const std::string &msg) @@ -528,7 +528,7 @@ bool GuildManager::afterRemove() localPlayer->clearGuilds(); } NotifyManager::notify(NotifyTypes::GUILD_LEFT); - delete2(mTab); + delete2(mTab) if (socialWindow != nullptr) socialWindow->removeTab(guild); diff --git a/src/particle/particle.cpp b/src/particle/particle.cpp index dacb7b1f2..dc0cacc56 100644 --- a/src/particle/particle.cpp +++ b/src/particle/particle.cpp @@ -98,7 +98,7 @@ Particle::~Particle() } // Delete child emitters and child particles clear(); - delete2(mAnimation); + delete2(mAnimation) if (mImage != nullptr) { if (mType == ParticleType::Image) diff --git a/src/progs/dyecmd/client.cpp b/src/progs/dyecmd/client.cpp index a905c326b..f31f2524a 100644 --- a/src/progs/dyecmd/client.cpp +++ b/src/progs/dyecmd/client.cpp @@ -443,15 +443,15 @@ void Client::gameClear() stopTimers(); - delete2(ipc); + delete2(ipc) if (logger != nullptr) logger->log1("Quitting2"); - delete2(mCurrentDialog); - delete2(popupManager); - delete2(dialogsManager); - delete2(gui); + delete2(mCurrentDialog) + delete2(popupManager) + delete2(dialogsManager) + delete2(gui) if (logger != nullptr) logger->log1("Quitting3"); @@ -484,7 +484,7 @@ void Client::gameClear() if (logger != nullptr) logger->log1("Quitting9"); - delete2(joystick); + delete2(joystick) keyboard.deinit(); @@ -521,7 +521,7 @@ void Client::gameClear() if (logger != nullptr) logger->log1("Quitting12"); -// delete2(chatLogger); +// delete2(chatLogger) TranslationManager::close(); VirtFs::deinit(); } @@ -646,7 +646,7 @@ int Client::gameExec() mOldState = mState; // Get rid of the dialog of the previous state - delete2(mCurrentDialog); + delete2(mCurrentDialog) // State has changed, while the quitDialog was active, it might // not be correct anymore diff --git a/src/progs/manaplus/actions/actions.cpp b/src/progs/manaplus/actions/actions.cpp index 452f0c368..785eed7ca 100644 --- a/src/progs/manaplus/actions/actions.cpp +++ b/src/progs/manaplus/actions/actions.cpp @@ -184,7 +184,7 @@ static int uploadUpdate(void *ptr, } } } -// delete2(info->upload); +// delete2(info->upload) info->upload = nullptr; delete info; return 0; diff --git a/src/progs/manaplus/actions/statusbar.cpp b/src/progs/manaplus/actions/statusbar.cpp index e15694902..52d5a0e51 100644 --- a/src/progs/manaplus/actions/statusbar.cpp +++ b/src/progs/manaplus/actions/statusbar.cpp @@ -55,7 +55,7 @@ namespace Actions impHandler0(switchQuickDrop) { - callYellowBarCond(changeQuickDropCounter) + callYellowBarCond(changeQuickDropCounter); } impHandler0(changeCrazyMove) diff --git a/src/progs/manaplus/client.cpp b/src/progs/manaplus/client.cpp index dabfe293b..fb239ba7f 100644 --- a/src/progs/manaplus/client.cpp +++ b/src/progs/manaplus/client.cpp @@ -651,7 +651,7 @@ void Client::gameClear() isTerminate = true; config.removeListeners(this); - delete2(assertListener); + delete2(assertListener) IPC::stop(); eventsManager.shutdown(); @@ -671,10 +671,10 @@ void Client::gameClear() if (charServerHandler != nullptr) charServerHandler->clear(); - delete2(ipc); + delete2(ipc) #ifdef USE_MUMBLE - delete2(mumbleManager); + delete2(mumbleManager) #endif // USE_MUMBLE PlayerInfo::deinit(); @@ -682,18 +682,18 @@ void Client::gameClear() // Before config.write() since it writes the shortcuts to the config for (unsigned f = 0; f < SHORTCUT_TABS; f ++) delete2(itemShortcut[f]) - delete2(emoteShortcut); - delete2(dropShortcut); + delete2(emoteShortcut) + delete2(dropShortcut) playerRelations.store(); if (logger != nullptr) logger->log1("Quitting2"); - delete2(mCurrentDialog); - delete2(popupManager); - delete2(dialogsManager); - delete2(gui); + delete2(mCurrentDialog) + delete2(popupManager) + delete2(dialogsManager) + delete2(gui) if (inventoryHandler != nullptr) inventoryHandler->clear(); @@ -735,7 +735,7 @@ void Client::gameClear() if (logger != nullptr) logger->log1("Quitting9"); - delete2(joystick); + delete2(joystick) keyboard.deinit(); @@ -772,7 +772,7 @@ void Client::gameClear() if (logger != nullptr) logger->log1("Quitting12"); - delete2(chatLogger); + delete2(chatLogger) TranslationManager::close(); } @@ -1049,7 +1049,7 @@ int Client::gameExec() if (mOldState == State::GAME) { - delete2(mGame); + delete2(mGame) assertListener = new AssertListener; Game::clearInstance(); ResourceManager::cleanOrphans(false); @@ -1081,7 +1081,7 @@ int Client::gameExec() mOldState = mState; // Get rid of the dialog of the previous state - delete2(mCurrentDialog); + delete2(mCurrentDialog) // State has changed, while the quitDialog was active, it might // not be correct anymore @@ -1395,16 +1395,16 @@ int Client::gameExec() chatLogger->setServerName(settings.serverName); #ifdef ANDROID - delete2(mCloseButton); + delete2(mCloseButton) #endif // ANDROID - delete2(mSetupButton); - delete2(mVideoButton); - delete2(mThemesButton); - delete2(mAboutButton); - delete2(mHelpButton); - delete2(mPerfomanceButton); - delete2(desktop); + delete2(mSetupButton) + delete2(mVideoButton) + delete2(mThemesButton) + delete2(mAboutButton) + delete2(mHelpButton) + delete2(mPerfomanceButton) + delete2(desktop) mCurrentDialog = nullptr; @@ -1846,7 +1846,7 @@ void Client::initTradeFilter() else { reportAlways("Error opening file for writing: %s", - tradeListName.c_str()); + tradeListName.c_str()) } tradeFile.close(); } @@ -2075,7 +2075,7 @@ void Client::runValidate() WindowManager::deleteValidateWindows(); unloadData(); - delete2(client); + delete2(client) VirtFs::deinit(); exit(0); } diff --git a/src/progs/manaplus/gui/viewport.cpp b/src/progs/manaplus/gui/viewport.cpp index d3ed684e2..83ddef480 100644 --- a/src/progs/manaplus/gui/viewport.cpp +++ b/src/progs/manaplus/gui/viewport.cpp @@ -504,7 +504,7 @@ bool Viewport::leftMouseAction() case ActorType::Avatar: default: reportAlways("Left click on unknown actor type: %d", - CAST_S32(type)); + CAST_S32(type)) break; } } diff --git a/src/render/mobileopengl2graphics.cpp b/src/render/mobileopengl2graphics.cpp index 73579005a..a89759d2b 100644 --- a/src/render/mobileopengl2graphics.cpp +++ b/src/render/mobileopengl2graphics.cpp @@ -75,7 +75,7 @@ var[vp + 20] = dstX + w; \ var[vp + 21] = dstY + h; \ var[vp + 22] = x2; \ - var[vp + 23] = y2; + var[vp + 23] = y2 #define toGL static_cast<GLfloat> @@ -128,7 +128,7 @@ MobileOpenGL2Graphics::~MobileOpenGL2Graphics() void MobileOpenGL2Graphics::deleteGLObjects() restrict2 { - delete2(mProgram); + delete2(mProgram) if (mVbo != 0U) mglDeleteBuffers(1, &mVbo); #ifndef __native_client__ diff --git a/src/render/mobileopenglgraphics.cpp b/src/render/mobileopenglgraphics.cpp index c8cbb7bba..7465ef3ff 100644 --- a/src/render/mobileopenglgraphics.cpp +++ b/src/render/mobileopenglgraphics.cpp @@ -68,7 +68,7 @@ vVar[vp + 8] = static_cast<GLshort>(dstX); \ vVar[vp + 9] = static_cast<GLshort>(dstY + h); \ vVar[vp + 10] = static_cast<GLshort>(dstX + w); \ - vVar[vp + 11] = static_cast<GLshort>(dstY + h); + vVar[vp + 11] = static_cast<GLshort>(dstY + h) GLuint MobileOpenGLGraphics::mTextureBinded = 0; #ifdef DEBUG_DRAW_CALLS diff --git a/src/render/modernopenglgraphics.cpp b/src/render/modernopenglgraphics.cpp index e742db33e..94efe61a4 100644 --- a/src/render/modernopenglgraphics.cpp +++ b/src/render/modernopenglgraphics.cpp @@ -77,7 +77,7 @@ var[vp + 20] = dstX + w; \ var[vp + 21] = dstY + h; \ var[vp + 22] = x2; \ - var[vp + 23] = y2; + var[vp + 23] = y2 GLuint ModernOpenGLGraphics::mTextureBinded = 0; #ifdef DEBUG_DRAW_CALLS @@ -125,7 +125,7 @@ ModernOpenGLGraphics::~ModernOpenGLGraphics() void ModernOpenGLGraphics::deleteGLObjects() restrict2 { - delete2(mProgram); + delete2(mProgram) if (mVbo != 0U) { // logger->log("delete buffer vbo: %u", mVbo); diff --git a/src/render/normalopenglgraphics.cpp b/src/render/normalopenglgraphics.cpp index edaeb17cb..f9f84a030 100644 --- a/src/render/normalopenglgraphics.cpp +++ b/src/render/normalopenglgraphics.cpp @@ -59,7 +59,7 @@ vVar[vp + 4] = dstX + w; \ vVar[vp + 5] = dstY + h; \ vVar[vp + 6] = dstX; \ - vVar[vp + 7] = dstY + h; + vVar[vp + 7] = dstY + h #define vertFillNv(tVar, vVar, srcX, srcY, dstX, dstY, w, h) \ @@ -78,7 +78,7 @@ vVar[vp + 4] = dstX + w; \ vVar[vp + 5] = dstY + h; \ vVar[vp + 6] = dstX; \ - vVar[vp + 7] = dstY + h; + vVar[vp + 7] = dstY + h namespace { diff --git a/src/render/openglx/mglxinit.cpp b/src/render/openglx/mglxinit.cpp index 74c378ed4..d73d981c6 100644 --- a/src/render/openglx/mglxinit.cpp +++ b/src/render/openglx/mglxinit.cpp @@ -30,13 +30,13 @@ void Glx::initFunctions() { - assignFunction(glXCreateContext); - assignFunction(glXGetCurrentContext); - assignFunctionARB(glXCreateContextAttribs); - assignFunction(glXChooseFBConfig); - assignFunction(glXDestroyContext); - assignFunction(glXMakeCurrent); - assignFunction(glXSwapBuffers); + assignFunction(glXCreateContext) + assignFunction(glXGetCurrentContext) + assignFunctionARB(glXCreateContextAttribs) + assignFunction(glXChooseFBConfig) + assignFunction(glXDestroyContext) + assignFunction(glXMakeCurrent) + assignFunction(glXSwapBuffers) } #endif // defined(USE_OPENGL) && defined(USE_X11) diff --git a/src/resources/animation/simpleanimation.cpp b/src/resources/animation/simpleanimation.cpp index 11ec9a01b..de9375489 100644 --- a/src/resources/animation/simpleanimation.cpp +++ b/src/resources/animation/simpleanimation.cpp @@ -67,7 +67,7 @@ SimpleAnimation::SimpleAnimation(XmlNodeConstPtr animationNode, SimpleAnimation::~SimpleAnimation() { - delete2(mAnimation); + delete2(mAnimation) if (mImageSet != nullptr) { mImageSet->decRef(); @@ -185,7 +185,7 @@ void SimpleAnimation::initializeAnimation(XmlNodeConstPtr animationNode, if (index < 0) { - reportAlways("No valid value for 'index'"); + reportAlways("No valid value for 'index'") continue; } @@ -193,7 +193,7 @@ void SimpleAnimation::initializeAnimation(XmlNodeConstPtr animationNode, if (img == nullptr) { - reportAlways("No image at index %d", index); + reportAlways("No image at index %d", index) continue; } @@ -207,7 +207,7 @@ void SimpleAnimation::initializeAnimation(XmlNodeConstPtr animationNode, if (start < 0 || end < 0) { - reportAlways("No valid value for 'start' or 'end'"); + reportAlways("No valid value for 'start' or 'end'") continue; } @@ -217,7 +217,7 @@ void SimpleAnimation::initializeAnimation(XmlNodeConstPtr animationNode, if (img == nullptr) { - reportAlways("No image at index %d", start); + reportAlways("No image at index %d", start) continue; } diff --git a/src/resources/atlas/atlasmanager.cpp b/src/resources/atlas/atlasmanager.cpp index 783e8b137..e1c847909 100644 --- a/src/resources/atlas/atlasmanager.cpp +++ b/src/resources/atlas/atlasmanager.cpp @@ -348,7 +348,7 @@ void AtlasManager::createSDLAtlas(TextureAtlas *const atlas) { reportAlways("Error creating surface for atlas. Size: %dx%d", width, - height); + height) BLOCK_END("AtlasManager::createSDLAtlas") return; } @@ -365,7 +365,7 @@ void AtlasManager::createSDLAtlas(TextureAtlas *const atlas) { reportAlways("Error converting surface to texture. Size: %dx%d", width, - height); + height) return; } diff --git a/src/resources/beinginfo.cpp b/src/resources/beinginfo.cpp index d7ffc2882..04c900538 100644 --- a/src/resources/beinginfo.cpp +++ b/src/resources/beinginfo.cpp @@ -215,8 +215,8 @@ void BeingInfo::addAttack(const int id, void BeingInfo::clear() { - delete2(unknown); - delete2(empty); + delete2(unknown) + delete2(empty) } void BeingInfo::init() diff --git a/src/resources/db/avatardb.cpp b/src/resources/db/avatardb.cpp index 3bc8f8a2c..4502b93f8 100644 --- a/src/resources/db/avatardb.cpp +++ b/src/resources/db/avatardb.cpp @@ -50,7 +50,7 @@ void AvatarDB::load() logger->log1("Initializing avatar database..."); loadXmlFile(paths.getStringValue("avatarsFile"), SkipError_false); loadXmlFile(paths.getStringValue("avatarsPatchFile"), SkipError_true); - loadXmlDir("avatarsPatchDir", loadXmlFile); + loadXmlDir("avatarsPatchDir", loadXmlFile) } void AvatarDB::loadXmlFile(const std::string &fileName, diff --git a/src/resources/db/clandb.cpp b/src/resources/db/clandb.cpp index c613067e3..52959e4f9 100644 --- a/src/resources/db/clandb.cpp +++ b/src/resources/db/clandb.cpp @@ -50,7 +50,7 @@ void ClanDb::load() logger->log1("Initializing clans database..."); loadXmlFile(paths.getStringValue("clansFile"), SkipError_false); loadXmlFile(paths.getStringValue("clansPatchFile"), SkipError_true); - loadXmlDir("clansPatchDir", loadXmlFile); + loadXmlDir("clansPatchDir", loadXmlFile) mLoaded = true; } @@ -90,7 +90,7 @@ void ClanDb::loadXmlFile(const std::string &fileName, ClanInfo *clanInfo = nullptr; if (mClansInfos.find(id) != mClansInfos.end()) { - reportAlways("ClanDb: Redefinition of clan ID %d", id); + reportAlways("ClanDb: Redefinition of clan ID %d", id) clanInfo = mClansInfos[id]; } if (clanInfo == nullptr) @@ -130,7 +130,7 @@ const ClanInfo *ClanDb::get(const int clanId) { reportAlways("ClanDb: Warning, unknown clan ID " "%d requested", - clanId); + clanId) return nullptr; } } diff --git a/src/resources/db/colordb.cpp b/src/resources/db/colordb.cpp index 236c4d934..401f67ed2 100644 --- a/src/resources/db/colordb.cpp +++ b/src/resources/db/colordb.cpp @@ -67,7 +67,7 @@ void ColorDB::load() SkipError_false); loadColorLists(paths.getStringValue("itemColorsPatchFile"), SkipError_true); - loadXmlDir("itemColorsPatchDir", loadColorLists); + loadXmlDir("itemColorsPatchDir", loadColorLists) it = mColorLists.find("hair"); if (it != mColorLists.end()) @@ -95,7 +95,7 @@ void ColorDB::loadHair(const std::string &fileName, return; } - reportAlways("Found legacy hair.xml"); + reportAlways("Found legacy hair.xml") for_each_xml_child_node(node, root) { if (xmlNameEqual(node, "include")) @@ -113,7 +113,7 @@ void ColorDB::loadHair(const std::string &fileName, if (colors.find(id) != colors.end()) { reportAlways("ColorDB: Redefinition of dye ID %d", - toInt(id, int)); + toInt(id, int)) } colors[id] = ItemColorData(id, XML::langProperty(node, "name", ""), @@ -194,7 +194,7 @@ std::string &ColorDB::getHairColorName(const ItemColor id) const ColorListsIterator it = mColorLists.find("hair"); if (it == mColorLists.end()) { - reportAlways("ColorDB: Error, hair colors list empty"); + reportAlways("ColorDB: Error, hair colors list empty") return mFail; } @@ -203,7 +203,7 @@ std::string &ColorDB::getHairColorName(const ItemColor id) if (i == (*it).second.end()) { reportAlways("ColorDB: Error, unknown dye ID# %d", - toInt(id, int)); + toInt(id, int)) return mFail; } return i->second.name; diff --git a/src/resources/db/commandsdb.cpp b/src/resources/db/commandsdb.cpp index ab98e7402..8195205d9 100644 --- a/src/resources/db/commandsdb.cpp +++ b/src/resources/db/commandsdb.cpp @@ -45,7 +45,7 @@ void CommandsDB::load() SkipError_false); loadXmlFile(paths.getStringValue("defaultCommandsPatchFile"), SkipError_true); - loadXmlDir("defaultCommandsPatchDir", loadXmlFile); + loadXmlDir("defaultCommandsPatchDir", loadXmlFile) mLoaded = true; } diff --git a/src/resources/db/deaddb.cpp b/src/resources/db/deaddb.cpp index cd921d05a..11f681bb1 100644 --- a/src/resources/db/deaddb.cpp +++ b/src/resources/db/deaddb.cpp @@ -44,7 +44,7 @@ void DeadDB::load() loadXmlFile(paths.getStringValue("deadMessagesFile"), SkipError_false); loadXmlFile(paths.getStringValue("deadMessagesPatchFile"), SkipError_true); - loadXmlDir("deadMessagesPatchDir", loadXmlFile); + loadXmlDir("deadMessagesPatchDir", loadXmlFile) mLoaded = true; } diff --git a/src/resources/db/elementaldb.cpp b/src/resources/db/elementaldb.cpp index 4263c930c..a1c55ef67 100644 --- a/src/resources/db/elementaldb.cpp +++ b/src/resources/db/elementaldb.cpp @@ -46,7 +46,7 @@ void ElementalDb::load() logger->log1("Initializing elemental database..."); loadXmlFile(paths.getStringValue("elementalsFile"), SkipError_false); loadXmlFile(paths.getStringValue("elementalsPatchFile"), SkipError_true); - loadXmlDir("elementalsPatchDir", loadXmlFile); + loadXmlDir("elementalsPatchDir", loadXmlFile) mLoaded = true; } @@ -145,7 +145,7 @@ BeingInfo *ElementalDb::get(const BeingTypeId id) { reportAlways("ElementalDb: Warning, unknown elemental ID " "%d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } } diff --git a/src/resources/db/emotedb.cpp b/src/resources/db/emotedb.cpp index 05808170a..65a1e0179 100644 --- a/src/resources/db/emotedb.cpp +++ b/src/resources/db/emotedb.cpp @@ -63,7 +63,7 @@ void EmoteDB::load() mLastEmote = 0; loadXmlFile(paths.getStringValue("emotesFile"), SkipError_false); loadXmlFile(paths.getStringValue("emotesPatchFile"), SkipError_true); - loadXmlDir("emotesPatchDir", loadXmlFile); + loadXmlDir("emotesPatchDir", loadXmlFile) loadSpecialXmlFile("graphics/sprites/manaplus_emotes.xml", SkipError_false); @@ -106,7 +106,7 @@ void EmoteDB::loadXmlFile(const std::string &fileName, if (id == -1) { reportAlways("Emote Database: Emote with missing ID in %s!", - paths.getStringValue("emotesFile").c_str()); + paths.getStringValue("emotesFile").c_str()) continue; } EmoteInfo *currentInfo = nullptr; @@ -179,7 +179,7 @@ void EmoteDB::loadSpecialXmlFile(const std::string &fileName, if (id == -1) { reportAlways("Emote Database: Emote with missing ID in " - "manaplus_emotes.xml!"); + "manaplus_emotes.xml!") continue; } const int altId = XML::getProperty(emoteNode, "altid", -1); @@ -263,7 +263,7 @@ const EmoteInfo *EmoteDB::get(const int id, const bool allowNull) if (allowNull) return nullptr; reportAlways("EmoteDB: Warning, unknown emote ID %d requested", - id); + id) return &mUnknown; } return i->second; @@ -282,7 +282,7 @@ const EmoteInfo *EmoteDB::get2(int id, const bool allowNull) if (allowNull) return nullptr; reportAlways("EmoteDB: Warning, unknown emote ID %d requested", - id); + id) return &mUnknown; } return i->second; diff --git a/src/resources/db/groupdb.cpp b/src/resources/db/groupdb.cpp index ea7ad0853..62db6c684 100644 --- a/src/resources/db/groupdb.cpp +++ b/src/resources/db/groupdb.cpp @@ -51,7 +51,7 @@ void GroupDb::load() loadXmlFile(paths.getStringValue("groupsFile"), SkipError_false); loadXmlFile(paths.getStringValue("groupsPatchFile"), SkipError_true); - loadXmlDir("groupsPatchDir", loadXmlFile); + loadXmlDir("groupsPatchDir", loadXmlFile) mLoaded = true; } @@ -75,7 +75,7 @@ static ServerCommandTypeT parseCommand(const std::string &str, { reportAlways("GroupsDb: unknown command name: '%s' in group id '%d'.", str.c_str(), - id); + id) } return ServerCommandType::Max; @@ -103,7 +103,7 @@ static ServerPermissionTypeT parsePermission(const std::string &str, reportAlways("GroupsDb: unknown permission name: " "'%s' in group id '%d'.", str.c_str(), - id); + id) } return ServerPermissionType::Max; @@ -137,7 +137,7 @@ static ServerCommandEnable::Type parseUseFlag(const std::string &str, reportAlways("GroupsDb: unknown use flag: '%s' in group id '%d'." "Possible values 'self', 'other', 'both'.", str.c_str(), - id); + id) return ServerCommandEnable::No; } } @@ -234,7 +234,7 @@ static void parseInherit(XmlNodePtr groupNode, { reportAlways("Unknown inherit group id '%d' in group '%d'", id2, - id); + id) continue; } GroupInfo *const groupInfo2 = (*it2).second; @@ -271,7 +271,7 @@ void GroupDb::loadXmlFile(const std::string &fileName, else { reportAlways("GroupsDb: Error while loading %s!", - fileName.c_str()); + fileName.c_str()) } return; } @@ -292,7 +292,7 @@ void GroupDb::loadXmlFile(const std::string &fileName, -1); if (id < 0) { - reportAlways("Empty id field in GroupDb"); + reportAlways("Empty id field in GroupDb") continue; } GroupInfosIter it = mGroups.find(id); @@ -300,7 +300,7 @@ void GroupDb::loadXmlFile(const std::string &fileName, if (it != mGroups.end()) { reportAlways("GroupDb: group with id %d already added", - id); + id) group = (*it).second; } else @@ -345,7 +345,7 @@ const std::string &GroupDb::getName(const int id) GroupInfos::const_iterator it = mGroups.find(id); if (it == mGroups.end()) { - reportAlways("Unknown group id requested: %d", id); + reportAlways("Unknown group id requested: %d", id) return mEmptyGroup.name; } return (*it).second->name; @@ -356,7 +356,7 @@ const std::string &GroupDb::getLongName(const int id) GroupInfos::const_iterator it = mGroups.find(id); if (it == mGroups.end()) { - reportAlways("Unknown group id requested: %d", id); + reportAlways("Unknown group id requested: %d", id) return mEmptyGroup.longName; } return (*it).second->longName; @@ -367,7 +367,7 @@ bool GroupDb::getShowBadge(const int id) GroupInfos::const_iterator it = mGroups.find(id); if (it == mGroups.end()) { - reportAlways("Unknown group id requested: %d", id); + reportAlways("Unknown group id requested: %d", id) return mEmptyGroup.showBadge; } return (*it).second->showBadge; @@ -378,7 +378,7 @@ bool GroupDb::getHighlightName(const int id) GroupInfos::const_iterator it = mGroups.find(id); if (it == mGroups.end()) { - reportAlways("Unknown group id requested: %d", id); + reportAlways("Unknown group id requested: %d", id) return mEmptyGroup.highlightName; } return (*it).second->highlightName; @@ -389,7 +389,7 @@ const std::string &GroupDb::getBadge(const int id) GroupInfos::const_iterator it = mGroups.find(id); if (it == mGroups.end()) { - reportAlways("Unknown group id requested: %d", id); + reportAlways("Unknown group id requested: %d", id) return mEmptyGroup.badge; } return (*it).second->badge; @@ -400,7 +400,7 @@ const GroupInfo *GroupDb::getGroup(const int id) GroupInfos::const_iterator it = mGroups.find(id); if (it == mGroups.end()) { - reportAlways("Unknown group id requested: %d", id); + reportAlways("Unknown group id requested: %d", id) return &mEmptyGroup; } return (*it).second; diff --git a/src/resources/db/homunculusdb.cpp b/src/resources/db/homunculusdb.cpp index 0d45a3864..1bbec6815 100644 --- a/src/resources/db/homunculusdb.cpp +++ b/src/resources/db/homunculusdb.cpp @@ -47,7 +47,7 @@ void HomunculusDB::load() logger->log1("Initializing homunculus database..."); loadXmlFile(paths.getStringValue("homunculusesFile"), SkipError_false); loadXmlFile(paths.getStringValue("homunculusesPatchFile"), SkipError_true); - loadXmlDir("homunculusesPatchDir", loadXmlFile); + loadXmlDir("homunculusesPatchDir", loadXmlFile) mLoaded = true; } @@ -151,7 +151,7 @@ BeingInfo *HomunculusDB::get(const BeingTypeId id) { reportAlways("HomunculusDB: Warning, unknown homunculus ID " "%d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } } diff --git a/src/resources/db/horsedb.cpp b/src/resources/db/horsedb.cpp index 56d646eba..f7d2b0173 100644 --- a/src/resources/db/horsedb.cpp +++ b/src/resources/db/horsedb.cpp @@ -77,7 +77,7 @@ void HorseDB::load() loadXmlFile(paths.getStringValue("horsesFile"), SkipError_false); loadXmlFile(paths.getStringValue("horsesPatchFile"), SkipError_true); - loadXmlDir("horsesPatchDir", loadXmlFile); + loadXmlDir("horsesPatchDir", loadXmlFile) mLoaded = true; } @@ -135,7 +135,7 @@ static void loadRiderOffset(XmlNodePtrConst node, if (dir == -1) { reportAlways("Wrong or missing horse rider direcion: %s", - dirName.c_str()); + dirName.c_str()) return; } HorseOffset &offset = currentInfo->offsets[dir]; @@ -180,7 +180,7 @@ void HorseDB::loadXmlFile(const std::string &fileName, if (id == -1) { reportAlways("Horse Database: Horse with missing ID in %s!", - paths.getStringValue("horsesFile").c_str()); + paths.getStringValue("horsesFile").c_str()) continue; } HorseInfo *currentInfo = nullptr; @@ -214,7 +214,7 @@ static void loadOffset(XmlNodePtrConst node, if (dir == -1) { reportAlways("Wrong or missing horse direcion: %s", - dirName.c_str()); + dirName.c_str()) return; } HorseOffset &offset = currentInfo->offsets[dir]; @@ -290,7 +290,7 @@ HorseInfo *HorseDB::get(const int id, const bool allowNull) if (allowNull) return nullptr; reportAlways("HorseDB: Warning, unknown horse ID %d requested", - id); + id) return &mUnknown; } return i->second; diff --git a/src/resources/db/itemdb.cpp b/src/resources/db/itemdb.cpp index 78ac033a6..5fbc16a79 100644 --- a/src/resources/db/itemdb.cpp +++ b/src/resources/db/itemdb.cpp @@ -304,7 +304,7 @@ void ItemDB::loadXmlFile(const std::string &fileName, if (id == 0) { reportAlways("ItemDB: Invalid or missing item ID in %s!", - fileName.c_str()); + fileName.c_str()) continue; } else if (mItemInfos.find(id) != mItemInfos.end()) @@ -403,7 +403,7 @@ void ItemDB::loadXmlFile(const std::string &fileName, { reportAlways("Inherit item %d from not existing item %d", id, - inherit); + inherit) } } @@ -682,7 +682,7 @@ void ItemDB::loadXmlFile(const std::string &fileName, if (attackRange == 0) { reportAlways("ItemDB: Missing attack range from weapon %i!", - id); + id) } } @@ -760,7 +760,7 @@ void ItemDB::unload() { logger->log1("Unloading item database..."); - delete2(mUnknown); + delete2(mUnknown) delete_all(mItemInfos); mItemInfos.clear(); @@ -798,7 +798,7 @@ const ItemInfo &ItemDB::get(const int id) if (i == mItemInfos.end()) { - reportAlways("ItemDB: Warning, unknown item ID# %d", id); + reportAlways("ItemDB: Warning, unknown item ID# %d", id) return *mUnknown; } @@ -818,7 +818,7 @@ const ItemInfo &ItemDB::get(const std::string &name) if (!name.empty()) { reportAlways("ItemDB: Warning, unknown item name \"%s\"", - name.c_str()); + name.c_str()) } return *mUnknown; } @@ -1000,7 +1000,7 @@ static void loadSoundRef(ItemInfo *const itemInfo, XmlNodeConstPtr node) else { reportAlways("ItemDB: Ignoring unknown sound event '%s'", - event.c_str()); + event.c_str()) } } diff --git a/src/resources/db/itemfielddb.cpp b/src/resources/db/itemfielddb.cpp index 76ad26036..ef289a1b4 100644 --- a/src/resources/db/itemfielddb.cpp +++ b/src/resources/db/itemfielddb.cpp @@ -47,7 +47,7 @@ void ItemFieldDb::load() loadXmlFile(paths.getStringValue("itemFieldsFile"), SkipError_false); loadXmlFile(paths.getStringValue("itemFieldsPatchFile"), SkipError_true); - loadXmlDir("itemFieldsPatchDir", loadXmlFile); + loadXmlDir("itemFieldsPatchDir", loadXmlFile) mLoaded = true; } @@ -65,7 +65,7 @@ static void loadFields(XmlNodeConstPtr groupNode, ""); if (name.empty()) { - reportAlways("Empty name field in ItemFieldDb"); + reportAlways("Empty name field in ItemFieldDb") continue; } const std::string description = XML::langProperty(node, @@ -73,7 +73,7 @@ static void loadFields(XmlNodeConstPtr groupNode, ""); if (description.empty()) { - reportAlways("Empty description field in ItemFieldDb"); + reportAlways("Empty description field in ItemFieldDb") continue; } const bool sign = XML::getBoolProperty(node, @@ -83,14 +83,14 @@ static void loadFields(XmlNodeConstPtr groupNode, { reportAlways( "Same field name detected in requeted and add groups: %s", - name.c_str()); + name.c_str()) continue; } if (fields1.find(name) != fields1.end()) { reportAlways( "Same field name detected: %s", - name.c_str()); + name.c_str()) continue; } fields1[name] = new ItemFieldType(name, diff --git a/src/resources/db/itemoptiondb.cpp b/src/resources/db/itemoptiondb.cpp index e983cd801..e66019a3d 100644 --- a/src/resources/db/itemoptiondb.cpp +++ b/src/resources/db/itemoptiondb.cpp @@ -45,7 +45,7 @@ void ItemOptionDb::load() logger->log1("Initializing item options database..."); loadXmlFile(paths.getStringValue("itemOptionsFile"), SkipError_false); loadXmlFile(paths.getStringValue("itemOptionsPatchFile"), SkipError_true); - loadXmlDir("itemOptionsPatchDir", loadXmlFile); + loadXmlDir("itemOptionsPatchDir", loadXmlFile) mLoaded = true; } @@ -99,7 +99,7 @@ void ItemOptionDb::loadXmlFile(const std::string &fileName, else { reportAlways("ItemFieldDb: Error while loading %s!", - fileName.c_str()); + fileName.c_str()) } return; } @@ -125,7 +125,7 @@ void ItemOptionDb::loadXmlFile(const std::string &fileName, 0); if (id <= 0) { - reportAlways("Empty id field in ItemOptionDb"); + reportAlways("Empty id field in ItemOptionDb") continue; } STD_VECTOR<ItemFieldType*> &options = mOptions[id]; diff --git a/src/resources/db/languagedb.cpp b/src/resources/db/languagedb.cpp index 92ae888de..5f162223a 100644 --- a/src/resources/db/languagedb.cpp +++ b/src/resources/db/languagedb.cpp @@ -42,7 +42,7 @@ void LanguageDb::load() logger->log1("Initializing languages database..."); loadXmlFile(paths.getStringValue("languagesFile"), SkipError_false); loadXmlFile(paths.getStringValue("languagesPatchFile"), SkipError_true); - loadXmlDir("languagesPatchDir", loadXmlFile); + loadXmlDir("languagesPatchDir", loadXmlFile) } void LanguageDb::loadXmlFile(const std::string &fileName, @@ -73,14 +73,14 @@ void LanguageDb::loadXmlFile(const std::string &fileName, const int id = XML::getProperty(node, "id", -1); if (id < 0) { - reportAlways("Missing lang id"); + reportAlways("Missing lang id") continue; } const std::string icon = XML::getProperty(node, "icon", ""); const std::string po = XML::getProperty(node, "po", ""); if (icon.empty()) { - reportAlways("LanguageDb: empty icon field"); + reportAlways("LanguageDb: empty icon field") } else { @@ -88,7 +88,7 @@ void LanguageDb::loadXmlFile(const std::string &fileName, } if (po.empty()) { - reportAlways("LanguageDb: empty po field"); + reportAlways("LanguageDb: empty po field") } else { diff --git a/src/resources/db/mapdb.cpp b/src/resources/db/mapdb.cpp index 971fa4578..818c50a9d 100644 --- a/src/resources/db/mapdb.cpp +++ b/src/resources/db/mapdb.cpp @@ -53,11 +53,11 @@ void MapDB::load() SkipError_true); loadRemapXmlFile(paths.getStringValue("mapsRemapPatchFile"), SkipError_true); - loadXmlDir("mapsRemapPatchDir", loadRemapXmlFile); + loadXmlDir("mapsRemapPatchDir", loadRemapXmlFile) loadInfo(paths.getStringValue("mapsFile"), SkipError_false); loadInfo(paths.getStringValue("mapsPatchFile"), SkipError_true); - loadXmlDir("mapsPatchDir", loadInfo); + loadXmlDir("mapsPatchDir", loadInfo) mLoaded = true; } diff --git a/src/resources/db/mercenarydb.cpp b/src/resources/db/mercenarydb.cpp index 72e6b3837..97ce1d014 100644 --- a/src/resources/db/mercenarydb.cpp +++ b/src/resources/db/mercenarydb.cpp @@ -47,7 +47,7 @@ void MercenaryDB::load() logger->log1("Initializing mercenary database..."); loadXmlFile(paths.getStringValue("mercenariesFile"), SkipError_false); loadXmlFile(paths.getStringValue("mercenariesPatchFile"), SkipError_true); - loadXmlDir("mercenariesPatchDir", loadXmlFile); + loadXmlDir("mercenariesPatchDir", loadXmlFile) mLoaded = true; } @@ -149,7 +149,7 @@ BeingInfo *MercenaryDB::get(const BeingTypeId id) { reportAlways("MercenaryDB: Warning, unknown mercenary ID " "%d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } } diff --git a/src/resources/db/moddb.cpp b/src/resources/db/moddb.cpp index 7e80fc393..b32264df7 100644 --- a/src/resources/db/moddb.cpp +++ b/src/resources/db/moddb.cpp @@ -45,7 +45,7 @@ void ModDB::load() logger->log1("Initializing mod database..."); loadXmlFile(paths.getStringValue("modsFile"), SkipError_false); loadXmlFile(paths.getStringValue("modsPatchFile"), SkipError_true); - loadXmlDir("modsPatchDir", loadXmlFile); + loadXmlDir("modsPatchDir", loadXmlFile) mLoaded = true; } diff --git a/src/resources/db/monsterdb.cpp b/src/resources/db/monsterdb.cpp index 68bc40703..3141c1cf5 100644 --- a/src/resources/db/monsterdb.cpp +++ b/src/resources/db/monsterdb.cpp @@ -51,7 +51,7 @@ void MonsterDB::load() logger->log1("Initializing monster database..."); loadXmlFile(paths.getStringValue("monstersFile"), SkipError_false); loadXmlFile(paths.getStringValue("monstersPatchFile"), SkipError_true); - loadXmlDir("monstersPatchDir", loadXmlFile); + loadXmlDir("monstersPatchDir", loadXmlFile) mLoaded = true; } @@ -152,7 +152,7 @@ BeingInfo *MonsterDB::get(const BeingTypeId id) if (i == mMonsterInfos.end()) { reportAlways("MonsterDB: Warning, unknown monster ID %d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } } diff --git a/src/resources/db/networkdb.cpp b/src/resources/db/networkdb.cpp index f44e94d75..3caebbc8a 100644 --- a/src/resources/db/networkdb.cpp +++ b/src/resources/db/networkdb.cpp @@ -44,7 +44,7 @@ void NetworkDb::load() logger->log1("Initializing network database..."); loadXmlFile(paths.getStringValue("networkFile"), SkipError_false); loadXmlFile(paths.getStringValue("networkPatchFile"), SkipError_true); - loadXmlDir("networkPatchDir", loadXmlFile); + loadXmlDir("networkPatchDir", loadXmlFile) mLoaded = true; } diff --git a/src/resources/db/npcdb.cpp b/src/resources/db/npcdb.cpp index 3f858ccc3..a647522d1 100644 --- a/src/resources/db/npcdb.cpp +++ b/src/resources/db/npcdb.cpp @@ -53,7 +53,7 @@ void NPCDB::load() loadXmlFile(paths.getStringValue("npcsFile"), SkipError_false); loadXmlFile(paths.getStringValue("npcsPatchFile"), SkipError_true); - loadXmlDir("npcsPatchDir", loadXmlFile); + loadXmlDir("npcsPatchDir", loadXmlFile) mLoaded = true; } @@ -92,7 +92,7 @@ void NPCDB::loadXmlFile(const std::string &fileName, if (id == BeingTypeId_zero) { reportAlways("NPC Database: NPC with missing ID in %s!", - paths.getStringValue("npcsFile").c_str()); + paths.getStringValue("npcsFile").c_str()) continue; } else if (mNPCInfos.find(id) != mNPCInfos.end()) @@ -127,7 +127,7 @@ void NPCDB::loadXmlFile(const std::string &fileName, { reportAlways("Not found currency '%s' for npc %d", currency.c_str(), - CAST_S32(id)); + CAST_S32(id)) } currentInfo->setCurrency(currency); @@ -191,7 +191,7 @@ BeingInfo *NPCDB::get(const BeingTypeId id) if (i == mNPCInfos.end()) { reportAlways("NPCDB: Warning, unknown NPC ID %d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } return i->second; diff --git a/src/resources/db/npcdialogdb.cpp b/src/resources/db/npcdialogdb.cpp index e2ec324ed..104884904 100644 --- a/src/resources/db/npcdialogdb.cpp +++ b/src/resources/db/npcdialogdb.cpp @@ -44,7 +44,7 @@ void NpcDialogDB::load() logger->log1("Loading npc dialog database..."); loadXmlFile(paths.getStringValue("npcDialogsFile"), SkipError_false); loadXmlFile(paths.getStringValue("npcDialogsPatchFile"), SkipError_true); - loadXmlDir("npcDialogsPatchDir", loadXmlFile); + loadXmlDir("npcDialogsPatchDir", loadXmlFile) mLoaded = true; } @@ -74,7 +74,7 @@ static void loadNpcDialogMenu(NpcDialogInfo *const dialog, button->image = XML::getProperty(childNode, "image", ""); if (button->name.empty() && button->image.empty()) { - reportAlways("Error: npc button without name or image"); + reportAlways("Error: npc button without name or image") delete button; continue; } @@ -89,7 +89,7 @@ static void loadNpcDialogMenu(NpcDialogInfo *const dialog, const std::string image = XML::getProperty(childNode, "image", ""); if (image.empty()) { - reportAlways("Error: no image attribute found in image tag."); + reportAlways("Error: no image attribute found in image tag.") continue; } NpcImageInfo *const imageInfo = new NpcImageInfo; @@ -105,7 +105,7 @@ static void loadNpcDialogMenu(NpcDialogInfo *const dialog, const std::string text = XML::getProperty(childNode, "text", ""); if (text.empty()) { - reportAlways("Error: no text attribute found in text tag."); + reportAlways("Error: no text attribute found in text tag.") continue; } NpcTextInfo *const textInfo = new NpcTextInfo; diff --git a/src/resources/db/palettedb.cpp b/src/resources/db/palettedb.cpp index 987f20389..fd61f20f5 100644 --- a/src/resources/db/palettedb.cpp +++ b/src/resources/db/palettedb.cpp @@ -60,7 +60,7 @@ void PaletteDB::loadPalette() } if (*it != "GIMP Palette") { - reportAlways("wrong GIMP palette file"); + reportAlways("wrong GIMP palette file") return; } ++ it; diff --git a/src/resources/db/petdb.cpp b/src/resources/db/petdb.cpp index c01e80d42..f1570ed08 100644 --- a/src/resources/db/petdb.cpp +++ b/src/resources/db/petdb.cpp @@ -49,7 +49,7 @@ void PETDB::load() logger->log1("Initializing PET database..."); loadXmlFile(paths.getStringValue("petsFile"), SkipError_false); loadXmlFile(paths.getStringValue("petsPatchFile"), SkipError_true); - loadXmlDir("petsPatchDir", loadXmlFile); + loadXmlDir("petsPatchDir", loadXmlFile) mLoaded = true; } @@ -88,7 +88,7 @@ void PETDB::loadXmlFile(const std::string &fileName, if (id == BeingTypeId_negOne) { reportAlways("PET Database: PET with missing ID in %s!", - paths.getStringValue("petsFile").c_str()); + paths.getStringValue("petsFile").c_str()) continue; } @@ -160,7 +160,7 @@ BeingInfo *PETDB::get(const BeingTypeId id) if (i == mPETInfos.end()) { reportAlways("PETDB: Warning, unknown PET ID %d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } return i->second; diff --git a/src/resources/db/questdb.cpp b/src/resources/db/questdb.cpp index 03d9bc31a..65d29f586 100644 --- a/src/resources/db/questdb.cpp +++ b/src/resources/db/questdb.cpp @@ -49,7 +49,7 @@ void QuestDb::load() logger->log1("Initializing quest database..."); loadXmlFile(paths.getStringValue("questsFile"), SkipError_false); loadXmlFile(paths.getStringValue("questsPatchFile"), SkipError_true); - loadXmlDir("questsPatchDir", loadXmlFile); + loadXmlDir("questsPatchDir", loadXmlFile) } static void loadQuest(const int var, diff --git a/src/resources/db/skillunitdb.cpp b/src/resources/db/skillunitdb.cpp index db7e3b2ed..130d40683 100644 --- a/src/resources/db/skillunitdb.cpp +++ b/src/resources/db/skillunitdb.cpp @@ -49,7 +49,7 @@ void SkillUnitDb::load() logger->log1("Initializing skill unit database..."); loadXmlFile(paths.getStringValue("skillUnitsFile"), SkipError_false); loadXmlFile(paths.getStringValue("skillUnitsPatchFile"), SkipError_true); - loadXmlDir("skillUnitsPatchDir", loadXmlFile); + loadXmlDir("skillUnitsPatchDir", loadXmlFile) mLoaded = true; } @@ -87,7 +87,7 @@ void SkillUnitDb::loadXmlFile(const std::string &fileName, if (id == BeingTypeId_negOne) { reportAlways("SkillUnitDb: skill unit with missing ID in %s!", - paths.getStringValue("skillUnitsFile").c_str()); + paths.getStringValue("skillUnitsFile").c_str()) continue; } @@ -154,7 +154,7 @@ BeingInfo *SkillUnitDb::get(const BeingTypeId id) { reportAlways("SkillUnitDb: Warning, unknown skill unit id " "%d requested", - toInt(id, int)); + toInt(id, int)) return BeingInfo::unknown; } return i->second; diff --git a/src/resources/db/sounddb.cpp b/src/resources/db/sounddb.cpp index 781f8915d..61149eb4e 100644 --- a/src/resources/db/sounddb.cpp +++ b/src/resources/db/sounddb.cpp @@ -42,7 +42,7 @@ void SoundDB::load() logger->log1("Initializing sound database..."); loadXmlFile(paths.getStringValue("soundsFile"), SkipError_false); loadXmlFile(paths.getStringValue("soundsPatchFile"), SkipError_true); - loadXmlDir("soundsPatchDir", loadXmlFile); + loadXmlDir("soundsPatchDir", loadXmlFile) } void SoundDB::loadXmlFile(const std::string &fileName, diff --git a/src/resources/db/statdb.cpp b/src/resources/db/statdb.cpp index 2757c0085..88300203f 100644 --- a/src/resources/db/statdb.cpp +++ b/src/resources/db/statdb.cpp @@ -92,7 +92,7 @@ void StatDb::load() loadXmlFile(paths.getStringValue("statFile"), SkipError_false); loadXmlFile(paths.getStringValue("statPatchFile"), SkipError_true); - loadXmlDir("statPatchDir", loadXmlFile); + loadXmlDir("statPatchDir", loadXmlFile) mLoaded = true; } @@ -112,7 +112,7 @@ static void loadBasicStats(XmlNodeConstPtr rootNode) { reportAlways("Wrong attr or id for basic " "stat with name %s", - name.c_str()); + name.c_str()) continue; } const std::string tag = XML::getProperty(node, "tag", ""); @@ -150,7 +150,7 @@ static void loadStats(XmlNodeConstPtr rootNode, { reportAlways("Wrong attr or id for extended " "stat with name %s", - name.c_str()); + name.c_str()) continue; } stats.push_back(BasicStat(static_cast<AttributesT>(id), @@ -207,7 +207,7 @@ void StatDb::loadXmlFile(const std::string &fileName, std::string page = XML::langProperty(node, "name", ""); if (page.empty()) { - reportAlways("Page without name in stats.xml"); + reportAlways("Page without name in stats.xml") page = "Unknown"; } loadStats(node, page); @@ -218,7 +218,7 @@ void StatDb::loadXmlFile(const std::string &fileName, if (mBasicStats.empty() && mStats.empty()) { - reportAlways("StatDb: no stats found"); + reportAlways("StatDb: no stats found") addDefaultStats(); } } diff --git a/src/resources/db/statuseffectdb.cpp b/src/resources/db/statuseffectdb.cpp index 9bc5e4e95..0eb7c27a6 100644 --- a/src/resources/db/statuseffectdb.cpp +++ b/src/resources/db/statuseffectdb.cpp @@ -52,7 +52,7 @@ StatusEffect *StatusEffectDB::getStatusEffect(const int index, if (it != effects.end()) return (*it).second; reportAlways("Missing status effect: %d", - index); + index) return nullptr; } @@ -66,7 +66,7 @@ void StatusEffectDB::load() loadXmlFile(paths.getStringValue("statusEffectsFile"), SkipError_false); loadXmlFile(paths.getStringValue("statusEffectsPatchFile"), SkipError_true); - loadXmlDir("statusEffectsPatchDir", loadXmlFile); + loadXmlDir("statusEffectsPatchDir", loadXmlFile) mLoaded = true; } diff --git a/src/resources/db/textdb.cpp b/src/resources/db/textdb.cpp index b6a902382..7304ac249 100644 --- a/src/resources/db/textdb.cpp +++ b/src/resources/db/textdb.cpp @@ -39,7 +39,7 @@ void TextDb::load() logger->log1("Initializing text database..."); loadXmlFile(paths.getStringValue("textsFile"), SkipError_false); loadXmlFile(paths.getStringValue("textsPatchFile"), SkipError_true); - loadXmlDir("textsPatchDir", loadXmlFile); + loadXmlDir("textsPatchDir", loadXmlFile) } void TextDb::loadXmlFile(const std::string &fileName, diff --git a/src/resources/db/unitsdb.cpp b/src/resources/db/unitsdb.cpp index b56be8fec..c9514b640 100644 --- a/src/resources/db/unitsdb.cpp +++ b/src/resources/db/unitsdb.cpp @@ -113,7 +113,7 @@ void UnitsDb::load() loadXmlFile(paths.getStringValue("unitsFile"), SkipError_false); loadXmlFile(paths.getStringValue("unitsPatchFile"), SkipError_true); - loadXmlDir("unitsPatchDir", loadXmlFile); + loadXmlDir("unitsPatchDir", loadXmlFile) } void UnitsDb::unload() @@ -186,7 +186,7 @@ static void loadCurrencies(XmlNodePtr parentNode) const std::string name = XML::getProperty(node, "name", ""); if (name.empty()) { - reportAlways("Error: unknown currency name."); + reportAlways("Error: unknown currency name.") continue; } mCurrencies[name] = loadUnit(node); diff --git a/src/resources/inventory/inventory.cpp b/src/resources/inventory/inventory.cpp index d013ef3f6..529a1b0b5 100644 --- a/src/resources/inventory/inventory.cpp +++ b/src/resources/inventory/inventory.cpp @@ -152,7 +152,7 @@ void Inventory::setItem(const int index, if (index < 0 || index >= CAST_S32(mSize)) { reportAlways("Warning: invalid inventory index: %d", - index); + index) return; } @@ -197,7 +197,7 @@ void Inventory::setCards(const int index, if (index < 0 || index >= CAST_S32(mSize)) { reportAlways("Warning: invalid inventory index: %d", - index); + index) return; } @@ -212,7 +212,7 @@ void Inventory::setOptions(const int index, if (index < 0 || index >= CAST_S32(mSize)) { reportAlways("Warning: invalid inventory index: %d", - index); + index) return; } Item *const item1 = mItems[index]; @@ -226,7 +226,7 @@ void Inventory::setTag(const int index, if (index < 0 || index >= CAST_S32(mSize)) { reportAlways("Warning: invalid inventory index: %d", - index); + index) return; } Item *const item1 = mItems[index]; @@ -254,7 +254,7 @@ void Inventory::removeItemAt(const int index) { if (mItems[index] == nullptr) return; - delete2(mItems[index]); + delete2(mItems[index]) mUsed--; if (mUsed < 0) // Already at 0, no need to distribute event mUsed = 0; diff --git a/src/resources/item/item.cpp b/src/resources/item/item.cpp index fb7290721..ff59fcd19 100644 --- a/src/resources/item/item.cpp +++ b/src/resources/item/item.cpp @@ -83,7 +83,7 @@ Item::~Item() mImage->decRef(); mImage = nullptr; } - delete2(mOptions); + delete2(mOptions) dragDrop.clearItem(this); } @@ -187,7 +187,7 @@ void Item::addCard(const int card) void Item::setOptions(const ItemOptionsList *const options) { - delete2(mOptions); + delete2(mOptions) mOptions = ItemOptionsList::copy(options); } diff --git a/src/resources/iteminfo.cpp b/src/resources/iteminfo.cpp index a1bd98bdb..1d2f8cfda 100644 --- a/src/resources/iteminfo.cpp +++ b/src/resources/iteminfo.cpp @@ -420,7 +420,7 @@ std::string ItemInfo::getColorName(const ItemColor idx) const { reportAlways("Color %d for palette %s not found", CAST_S32(idx), - mColorsListName.c_str()); + mColorsListName.c_str()) return std::string(); } return it->second.name; @@ -437,7 +437,7 @@ std::string ItemInfo::getColor(const ItemColor idx) const { reportAlways("Color %d for palette %s not found", CAST_S32(idx), - mColorsListName.c_str()); + mColorsListName.c_str()) return std::string(); } return it->second.color; @@ -454,7 +454,7 @@ std::string ItemInfo::getIconColorName(const ItemColor idx) const { reportAlways("Color %d for palette %s not found", CAST_S32(idx), - mColorsListName.c_str()); + mColorsListName.c_str()) return std::string(); } return it->second.name; @@ -471,7 +471,7 @@ std::string ItemInfo::getIconColor(const ItemColor idx) const { reportAlways("Color %d for palette %s not found", CAST_S32(idx), - mColorsListName.c_str()); + mColorsListName.c_str()) return std::string(); } return it->second.color; diff --git a/src/resources/loaders/atlasloader.cpp b/src/resources/loaders/atlasloader.cpp index aa2585b27..fd435e57f 100644 --- a/src/resources/loaders/atlasloader.cpp +++ b/src/resources/loaders/atlasloader.cpp @@ -47,7 +47,7 @@ struct AtlasLoader final AtlasResource *const resource = AtlasManager::loadTextureAtlas( rl->name, *rl->files); if (resource == nullptr) - reportAlways("Atlas creation error: %s", rl->name.c_str()); + reportAlways("Atlas creation error: %s", rl->name.c_str()) return resource; } }; diff --git a/src/resources/loaders/emptyatlasloader.cpp b/src/resources/loaders/emptyatlasloader.cpp index b3d943075..71ea865e9 100644 --- a/src/resources/loaders/emptyatlasloader.cpp +++ b/src/resources/loaders/emptyatlasloader.cpp @@ -48,7 +48,7 @@ struct EmptyAtlasLoader final AtlasResource *const resource = AtlasManager::loadEmptyAtlas( rl->name, *rl->files); if (resource == nullptr) - reportAlways("Empty atlas creation error: %s", rl->name.c_str()); + reportAlways("Empty atlas creation error: %s", rl->name.c_str()) return resource; } }; diff --git a/src/resources/loaders/imageloader.cpp b/src/resources/loaders/imageloader.cpp index 2c619a4b8..4c573e98d 100644 --- a/src/resources/loaders/imageloader.cpp +++ b/src/resources/loaders/imageloader.cpp @@ -67,7 +67,7 @@ namespace if (rw == nullptr) { delete d; - reportAlways("Image loading error: %s", path1.c_str()); + reportAlways("Image loading error: %s", path1.c_str()) BLOCK_END("DyedImageLoader::load") return nullptr; } @@ -75,7 +75,7 @@ namespace : imageHelper->load(rw); delete d; if (res == nullptr) - reportAlways("Image loading error: %s", path1.c_str()); + reportAlways("Image loading error: %s", path1.c_str()) BLOCK_END("DyedImageLoader::load") return res; } diff --git a/src/resources/loaders/imagesetloader.cpp b/src/resources/loaders/imagesetloader.cpp index 4fbcc2fc2..d62d3727b 100644 --- a/src/resources/loaders/imagesetloader.cpp +++ b/src/resources/loaders/imagesetloader.cpp @@ -53,7 +53,7 @@ struct ImageSetLoader final Image *const img = Loader::getImage(rl->path); if (img == nullptr) { - reportAlways("Image loading error: %s", rl->path.c_str()); + reportAlways("Image loading error: %s", rl->path.c_str()) return nullptr; } ImageSet *const res = new ImageSet(img, rl->w, rl->h, 0, 0); diff --git a/src/resources/loaders/musicloader.cpp b/src/resources/loaders/musicloader.cpp index 6bd906490..97b5ceb73 100644 --- a/src/resources/loaders/musicloader.cpp +++ b/src/resources/loaders/musicloader.cpp @@ -51,7 +51,7 @@ namespace if (rw == nullptr) { reportAlways("Error loading resource: %s", - rl->path.c_str()); + rl->path.c_str()) return nullptr; } if (Mix_Music *const music = SDL::LoadMUSOgg_RW(rw)) diff --git a/src/resources/loaders/rescaledloader.cpp b/src/resources/loaders/rescaledloader.cpp index 085614870..8f872943b 100644 --- a/src/resources/loaders/rescaledloader.cpp +++ b/src/resources/loaders/rescaledloader.cpp @@ -53,7 +53,7 @@ namespace if (rescaled == nullptr) { reportAlways("Rescale image failed: %s", - rl->image->mIdPath.c_str()); + rl->image->mIdPath.c_str()) return nullptr; } return rescaled; diff --git a/src/resources/loaders/shaderloader.cpp b/src/resources/loaders/shaderloader.cpp index 5f1edd3a7..ca3cd6a13 100644 --- a/src/resources/loaders/shaderloader.cpp +++ b/src/resources/loaders/shaderloader.cpp @@ -51,7 +51,7 @@ namespace = static_cast<const ShaderLoader *>(v); Shader *const resource = shaders.createShader(rl->type, rl->name); if (resource == nullptr) - reportAlways("Shader creation error: %s", rl->name.c_str()); + reportAlways("Shader creation error: %s", rl->name.c_str()) return resource; } }; diff --git a/src/resources/loaders/shaderprogramloader.cpp b/src/resources/loaders/shaderprogramloader.cpp index d0c6f1f41..2f981f590 100644 --- a/src/resources/loaders/shaderprogramloader.cpp +++ b/src/resources/loaders/shaderprogramloader.cpp @@ -55,7 +55,7 @@ namespace rl->fragment, rl->isNewShader); if (resource == nullptr) - reportAlways("Shader program creation error"); + reportAlways("Shader program creation error") return resource; } }; diff --git a/src/resources/loaders/soundloader.cpp b/src/resources/loaders/soundloader.cpp index 96c90d4b4..f8550f631 100644 --- a/src/resources/loaders/soundloader.cpp +++ b/src/resources/loaders/soundloader.cpp @@ -50,7 +50,7 @@ namespace if (rw == nullptr) { reportAlways("Error loading resource: %s", - rl->path.c_str()); + rl->path.c_str()) return nullptr; } // Load the music data and free the RWops structure @@ -61,7 +61,7 @@ namespace return new SoundEffect(tmpSoundEffect, rl->path); } reportAlways("Error, failed to load sound effect: %s", - SDL_GetError()); + SDL_GetError()) return nullptr; } }; diff --git a/src/resources/loaders/subimageloader.cpp b/src/resources/loaders/subimageloader.cpp index 3b8672821..6abba11c9 100644 --- a/src/resources/loaders/subimageloader.cpp +++ b/src/resources/loaders/subimageloader.cpp @@ -57,7 +57,7 @@ namespace if (res == nullptr) { reportAlways("SubImage loading error: %s", - rl->parent->mSource.c_str()); + rl->parent->mSource.c_str()) } return res; } diff --git a/src/resources/loaders/walklayerloader.cpp b/src/resources/loaders/walklayerloader.cpp index 3fcff35c2..c4bd99c44 100644 --- a/src/resources/loaders/walklayerloader.cpp +++ b/src/resources/loaders/walklayerloader.cpp @@ -49,7 +49,7 @@ struct WalkLayerLoader final WalkLayerLoader *>(v); Resource *const resource = NavigationManager::loadWalkLayer(rl->map); if (resource == nullptr) - reportAlways("WalkLayer creation error"); + reportAlways("WalkLayer creation error") return resource; } }; diff --git a/src/resources/map/map.cpp b/src/resources/map/map.cpp index bcec1a4a3..3cc6ecafe 100644 --- a/src/resources/map/map.cpp +++ b/src/resources/map/map.cpp @@ -191,9 +191,9 @@ Map::~Map() delete_all(mForegrounds); delete_all(mBackgrounds); delete_all(mTileAnimations); - delete2(mSpecialLayer); - delete2(mTempLayer); - delete2(mObjects); + delete2(mSpecialLayer) + delete2(mTempLayer) + delete2(mObjects) delete_all(mMapPortals); #ifdef USE_OPENGL if (mAtlas != nullptr) @@ -202,7 +202,7 @@ Map::~Map() mAtlas = nullptr; } #endif // USE_OPENGL - delete2(mHeights); + delete2(mHeights) delete [] mMetaTiles; } @@ -601,12 +601,12 @@ void Map::drawCollision(Graphics *restrict const graphics, int width = 0; const int x0 = x; - fillCollision(BlockMask::WALL, COLLISION_HIGHLIGHT); - fillCollision(BlockMask::AIR, AIR_COLLISION_HIGHLIGHT); - fillCollision(BlockMask::WATER, WATER_COLLISION_HIGHLIGHT); - fillCollision(BlockMask::GROUNDTOP, GROUNDTOP_COLLISION_HIGHLIGHT); - fillCollision(BlockMask::PLAYERWALL, COLLISION_HIGHLIGHT); - fillCollision(BlockMask::MONSTERWALL, MONSTER_COLLISION_HIGHLIGHT); + fillCollision(BlockMask::WALL, COLLISION_HIGHLIGHT) + fillCollision(BlockMask::AIR, AIR_COLLISION_HIGHLIGHT) + fillCollision(BlockMask::WATER, WATER_COLLISION_HIGHLIGHT) + fillCollision(BlockMask::GROUNDTOP, GROUNDTOP_COLLISION_HIGHLIGHT) + fillCollision(BlockMask::PLAYERWALL, COLLISION_HIGHLIGHT) + fillCollision(BlockMask::MONSTERWALL, MONSTER_COLLISION_HIGHLIGHT) } } } @@ -1241,7 +1241,7 @@ void Map::saveExtraLayer() const restrict2 if (!mapFile.is_open()) { reportAlways("Error opening file for writing: %s", - mapFileName.c_str()); + mapFileName.c_str()) return; } diff --git a/src/resources/map/tileanimation.cpp b/src/resources/map/tileanimation.cpp index f31fed7ae..6f6ee9e3e 100644 --- a/src/resources/map/tileanimation.cpp +++ b/src/resources/map/tileanimation.cpp @@ -40,7 +40,7 @@ TileAnimation::TileAnimation(Animation *const ani) : TileAnimation::~TileAnimation() { - delete2(mAnimation); + delete2(mAnimation) } bool TileAnimation::update(const int ticks) diff --git a/src/resources/mapreader.cpp b/src/resources/mapreader.cpp index 74059a034..eac985d02 100644 --- a/src/resources/mapreader.cpp +++ b/src/resources/mapreader.cpp @@ -203,19 +203,19 @@ int inflateMemory(unsigned char *restrict const in, { if (ret == Z_MEM_ERROR) { - reportAlways("Error: Out of memory while decompressing map data!"); + reportAlways("Error: Out of memory while decompressing map data!") } else if (ret == Z_VERSION_ERROR) { - reportAlways("Error: Incompatible zlib version!"); + reportAlways("Error: Incompatible zlib version!") } else if (ret == Z_DATA_ERROR) { - reportAlways("Error: Incorrect zlib compressed data!"); + reportAlways("Error: Incorrect zlib compressed data!") } else { - reportAlways("Error: Unknown error while decompressing map data!"); + reportAlways("Error: Unknown error while decompressing map data!") } free(out); @@ -284,7 +284,7 @@ Map *MapReader::readMap(const std::string &restrict filename, else { reportAlways("Error while parsing map file (%s)!", - realFilename.c_str()); + realFilename.c_str()) } if (map != nullptr) @@ -306,7 +306,7 @@ Map *MapReader::readMap(const std::string &restrict filename, void MapReader::loadLayers(const std::string &path) { BLOCK_START("MapReader::loadLayers") - loadXmlDir2(path, addLayerToList, ".tmx", SkipError_false); + loadXmlDir2(path, addLayerToList, ".tmx", SkipError_false) BLOCK_END("MapReader::loadLayers") } @@ -348,7 +348,7 @@ Map *MapReader::readMap(XmlNodePtrConst node, const std::string &path) { reportAlways("MapReader: Warning: " "Uninitialized tile width or height value for map: %s", - path.c_str()); + path.c_str()) BLOCK_END("MapReader::readMap xml") return nullptr; } @@ -377,7 +377,7 @@ Map *MapReader::readMap(XmlNodePtrConst node, const std::string &path) else { reportAlways("Missing atlas for map: %s", - fileName.c_str()); + fileName.c_str()) } } BLOCK_END("MapReader::readMap load atlas") @@ -674,7 +674,7 @@ bool MapReader::readBase64Layer(XmlNodeConstPtrConst childNode, && compression != "zlib") { reportAlways("Warning: only gzip and zlib layer" - " compression supported!"); + " compression supported!") return false; } @@ -730,7 +730,7 @@ bool MapReader::readBase64Layer(XmlNodeConstPtrConst childNode, if (inflated == nullptr) { - reportAlways("Error: Could not decompress layer!"); + reportAlways("Error: Could not decompress layer!") return false; } } @@ -1143,14 +1143,14 @@ Tileset *MapReader::readTileset(XmlNodePtr node, graphicsManager.getUseAtlases()) { reportAlways("Error: image '%s' not present in atlas", - source.c_str()); + source.c_str()) } #endif // USE_OPENGL } else { reportAlways("Error: Failed to load tileset (%s)", - source.c_str()); + source.c_str()) } } } diff --git a/src/resources/openglimagehelper.cpp b/src/resources/openglimagehelper.cpp index a2aa72958..ff3a2e4f3 100644 --- a/src/resources/openglimagehelper.cpp +++ b/src/resources/openglimagehelper.cpp @@ -69,7 +69,7 @@ Image *OpenGLImageHelper::load(SDL_RWops *const rw, Dye const &dye) SDL_Surface *const tmpImage = loadPng(rw); if (tmpImage == nullptr) { - reportAlways("Error, image load failed: %s", SDL_GetError()); + reportAlways("Error, image load failed: %s", SDL_GetError()) return nullptr; } @@ -148,7 +148,7 @@ SDL_Surface *OpenGLImageHelper::convertSurfaceNormalize(SDL_Surface *tmpImage, if (realWidth < width || realHeight < height) { reportAlways("Warning: image too large, cropping to %dx%d texture!", - tmpImage->w, tmpImage->h); + tmpImage->w, tmpImage->h) } #ifdef USE_SDL2 @@ -190,7 +190,7 @@ SDL_Surface *OpenGLImageHelper::convertSurfaceNormalize(SDL_Surface *tmpImage, if (tmpImage == nullptr) { - reportAlways("Error, image convert failed: out of memory"); + reportAlways("Error, image convert failed: out of memory") return nullptr; } SDL_BlitSurface(oldImage, nullptr, tmpImage, nullptr); @@ -242,7 +242,7 @@ SDL_Surface *OpenGLImageHelper::convertSurface(SDL_Surface *tmpImage, if (tmpImage == nullptr) { - reportAlways("Error, image convert failed: out of memory"); + reportAlways("Error, image convert failed: out of memory") return nullptr; } SDL_BlitSurface(oldImage, nullptr, tmpImage, nullptr); @@ -298,7 +298,7 @@ void OpenGLImageHelper::bindTexture(const GLuint texture) case RENDER_NULL: case RENDER_LAST: default: - reportAlways("Unknown OpenGL backend: %d", mUseOpenGL); + reportAlways("Unknown OpenGL backend: %d", mUseOpenGL) break; } } @@ -416,7 +416,7 @@ Image *OpenGLImageHelper::glLoad(SDL_Surface *tmpImage, { std::string errmsg = GraphicsManager::errorToString(error); reportAlways("Error: Image GL import failed: %s (%u)", - errmsg.c_str(), error); + errmsg.c_str(), error) // return nullptr; } diff --git a/src/resources/resourcemanager/resourcemanager.cpp b/src/resources/resourcemanager/resourcemanager.cpp index 0a28ac83a..db175ea20 100644 --- a/src/resources/resourcemanager/resourcemanager.cpp +++ b/src/resources/resourcemanager/resourcemanager.cpp @@ -96,7 +96,7 @@ void deleteResourceManager() #ifdef UNITTESTS if (status) - reportAlways("Found leaked resources."); + reportAlways("Found leaked resources.") #endif // UNITTESTS iter = mResources.begin(); @@ -447,7 +447,7 @@ Resource *get(const std::string &idPath, } else { - reportAlways("Error loading image: %s", idPath.c_str()); + reportAlways("Error loading image: %s", idPath.c_str()) } #else // DISABLE_RESOURCE_CACHING @@ -464,7 +464,7 @@ Resource *get(const std::string &idPath, } else { - reportAlways("Error loading image: " + idPath); + reportAlways("Error loading image: " + idPath) } #endif // DISABLE_RESOURCE_CACHING @@ -493,7 +493,7 @@ void release(Resource *const res) { // +++ need reenable after Resource will have type field // reportAlways("no resource in cache: %s", -// res->mIdPath.c_str()); +// res->mIdPath.c_str()) delete res; return; } @@ -501,7 +501,7 @@ void release(Resource *const res) { // +++ need reenable after Resource will have type field // reportAlways("in cache other image: %s", -// res->mIdPath.c_str()); +// res->mIdPath.c_str()) delete res; return; } @@ -619,7 +619,7 @@ void deleteInstance() } #ifdef UNITTESTS if (status) - reportAlways("Found leaked resources."); + reportAlways("Found leaked resources.") #endif // UNITTESTS #endif // DUMP_LEAKED_RESOURCES diff --git a/src/resources/sdl2imagehelper.cpp b/src/resources/sdl2imagehelper.cpp index 0875a341f..3340032b7 100644 --- a/src/resources/sdl2imagehelper.cpp +++ b/src/resources/sdl2imagehelper.cpp @@ -71,7 +71,7 @@ Image *SDLImageHelper::_SDLload(SDL_Surface *tmpImage) if (!texture) { reportAlways("Texture from surface creation failed: %s", - SDL_GetError()); + SDL_GetError()) return nullptr; } SDL_SetTextureBlendMode(texture, SDL_BLENDMODE_BLEND); diff --git a/src/resources/sdlgfxblitfunc.cpp b/src/resources/sdlgfxblitfunc.cpp index d72e7fbc5..fa74c82c3 100644 --- a/src/resources/sdlgfxblitfunc.cpp +++ b/src/resources/sdlgfxblitfunc.cpp @@ -383,7 +383,7 @@ static void _SDL_gfxBlitBlitterRGBA(const SDL_gfxBlitInfo *const info) src += 4; dst += 4; - }, width); + }, width) src += srcskip; dst += dstskip; } @@ -451,7 +451,7 @@ int SDLgfxBlitRGBA(SDL_Surface *const src, if (src == nullptr || dst == nullptr) { - reportAlways("SDLgfxBlitRGBA: passed a NULL surface"); + reportAlways("SDLgfxBlitRGBA: passed a NULL surface") return -1; } diff --git a/src/resources/sdlimagehelper.cpp b/src/resources/sdlimagehelper.cpp index 2c5970a49..0ed5c9f52 100644 --- a/src/resources/sdlimagehelper.cpp +++ b/src/resources/sdlimagehelper.cpp @@ -57,7 +57,7 @@ Image *SDLImageHelper::load(SDL_RWops *const rw, Dye const &dye) if (tmpImage == nullptr) { reportAlways("Error, image load failed: %s", - SDL_GetError()); + SDL_GetError()) return nullptr; } @@ -196,7 +196,7 @@ Image *SDLImageHelper::createTextSurface(SDL_Surface *const tmpImage, if (image == nullptr) { - reportAlways("Error: Image convert failed."); + reportAlways("Error: Image convert failed.") delete [] alphaChannel; return nullptr; } @@ -224,7 +224,7 @@ Image *SDLImageHelper::_SDLload(SDL_Surface *tmpImage) if (tmpImage->format->BitsPerPixel != 32) { - reportAlways("Non 32 bit image detected"); + reportAlways("Non 32 bit image detected") tmpImage = convertTo32Bit(tmpImage); if (tmpImage == nullptr) @@ -296,7 +296,7 @@ Image *SDLImageHelper::_SDLload(SDL_Surface *tmpImage) if (image == nullptr) { - reportAlways("Error: Image convert failed."); + reportAlways("Error: Image convert failed.") delete [] alphaChannel; return nullptr; } diff --git a/src/resources/sprite/animatedsprite.cpp b/src/resources/sprite/animatedsprite.cpp index 965cffbec..9e6e2e327 100644 --- a/src/resources/sprite/animatedsprite.cpp +++ b/src/resources/sprite/animatedsprite.cpp @@ -134,7 +134,7 @@ AnimatedSprite::~AnimatedSprite() { mDelayLoad->clearSprite(); DelayedManager::removeDelayLoad(mDelayLoad); - delete2(mDelayLoad); + delete2(mDelayLoad) } } diff --git a/src/resources/sprite/spritedef.cpp b/src/resources/sprite/spritedef.cpp index b2a53fd93..1e38c4ca5 100644 --- a/src/resources/sprite/spritedef.cpp +++ b/src/resources/sprite/spritedef.cpp @@ -105,7 +105,7 @@ SpriteDef *SpriteDef::load(const std::string &animationFile, if ((rootNode == nullptr) || !xmlNameEqual(rootNode, "sprite")) { reportAlways("Error, failed to parse sprite %s", - animationFile.c_str()); + animationFile.c_str()) const std::string errorFile = pathJoin(paths.getStringValue("sprites"), paths.getStringValue("spriteErrorFile")); BLOCK_END("SpriteDef::load") @@ -256,7 +256,7 @@ void SpriteDef::loadImageSet(XmlNodeConstPtr node, { reportAlways("%s: Couldn't load imageset: %s", mSource.c_str(), - imageSrc.c_str()); + imageSrc.c_str()) return; } @@ -273,7 +273,7 @@ const ImageSet *SpriteDef::getImageSet(const std::string &imageSetName) const reportAlways("%s: Imageset \"%s\" not defined in %s", mSource.c_str(), imageSetName.c_str(), - mIdPath.c_str()); + mIdPath.c_str()) return nullptr; } return si->second; @@ -295,7 +295,7 @@ void SpriteDef::loadAction(XmlNodeConstPtr node, reportAlways("%s: Unknown action \"%s\" defined in %s", mSource.c_str(), actionName.c_str(), - mIdPath.c_str()); + mIdPath.c_str()) return; } Action *const action = new Action(actionName); @@ -305,7 +305,7 @@ void SpriteDef::loadAction(XmlNodeConstPtr node, // dirty hack to fix bad resources in tmw server if (actionName == "attack_stab") { - reportAlways("Found legacy attribute attack_stab in animation"); + reportAlways("Found legacy attribute attack_stab in animation") addAction(hp, "attack", action); } @@ -345,7 +345,7 @@ void SpriteDef::loadAnimation(XmlNodeConstPtr animationNode, reportAlways("%s: Unknown direction \"%s\" used in %s", mSource.c_str(), directionName.c_str(), - mIdPath.c_str()); + mIdPath.c_str()) return; } @@ -383,7 +383,7 @@ void SpriteDef::loadAnimation(XmlNodeConstPtr animationNode, reportAlways( "%s: No valid value for 'index' at direction '%s'", mSource.c_str(), - directionName.c_str()); + directionName.c_str()) continue; } @@ -393,7 +393,7 @@ void SpriteDef::loadAnimation(XmlNodeConstPtr animationNode, reportAlways("%s: No image at index %d at direction '%s'", mSource.c_str(), index + variant_offset, - directionName.c_str()); + directionName.c_str()) continue; } @@ -411,7 +411,7 @@ void SpriteDef::loadAnimation(XmlNodeConstPtr animationNode, { reportAlways("%s: No valid value for 'repeat' at direction %s", mSource.c_str(), - directionName.c_str()); + directionName.c_str()) continue; } @@ -495,7 +495,7 @@ void SpriteDef::includeSprite(XmlNodeConstPtr includeNode, const int variant) { reportAlways("%s: Tried to include %s which already is included.", mSource.c_str(), - filename.c_str()); + filename.c_str()) return; } mProcessedFiles.insert(filename); @@ -511,7 +511,7 @@ void SpriteDef::includeSprite(XmlNodeConstPtr includeNode, const int variant) { reportAlways("%s: No sprite root node in %s", mSource.c_str(), - filename.c_str()); + filename.c_str()) doc->decRef(); return; } @@ -599,7 +599,7 @@ bool SpriteDef::addSequence(const int start, if (start < 0 || end < 0) { reportAlways("%s: No valid value for 'start' or 'end'", - mSource.c_str()); + mSource.c_str()) return true; } @@ -616,7 +616,7 @@ bool SpriteDef::addSequence(const int start, { reportAlways("%s: No image at index %d", mSource.c_str(), - pos + variant_offset); + pos + variant_offset) pos ++; continue; } @@ -641,7 +641,7 @@ bool SpriteDef::addSequence(const int start, { reportAlways("%s: No image at index %d", mSource.c_str(), - pos + variant_offset); + pos + variant_offset) pos ++; continue; } diff --git a/src/soundmanager.cpp b/src/soundmanager.cpp index 358b8e17c..2922bf66f 100644 --- a/src/soundmanager.cpp +++ b/src/soundmanager.cpp @@ -264,7 +264,7 @@ void SoundManager::info() uint16_t audioFormat = 0; int channels = 0; - MIX_VERSION(&compiledVersion); + MIX_VERSION(&compiledVersion) const SDL_version *const linkedVersion = Mix_Linked_Version(); #ifdef USE_SDL2 @@ -342,7 +342,7 @@ static SDLMusic *loadMusic(const std::string &fileName, if (!VirtFs::exists(path)) { if (skipError == SkipError_false) - reportAlways("Music file not found: %s", fileName.c_str()); + reportAlways("Music file not found: %s", fileName.c_str()) return nullptr; } return Loader::getMusic(path); diff --git a/src/spellmanager.cpp b/src/spellmanager.cpp index 34fe739a5..dcf7770fe 100644 --- a/src/spellmanager.cpp +++ b/src/spellmanager.cpp @@ -324,7 +324,7 @@ void SpellManager::load() if (!var##method.empty()) \ serverConfig.setValue((str) + toString(i), var##method); \ else \ - serverConfig.deleteKey((str) + toString(i)); + serverConfig.deleteKey((str) + toString(i)) void SpellManager::save() const { diff --git a/src/test/testmain.cpp b/src/test/testmain.cpp index 43c2c3f62..db9373e84 100644 --- a/src/test/testmain.cpp +++ b/src/test/testmain.cpp @@ -48,7 +48,7 @@ TestMain::TestMain() : TestMain::~TestMain() { - delete2(log); + delete2(log) } void TestMain::initConfig() diff --git a/src/text.cpp b/src/text.cpp index 88f56b325..d1ee14287 100644 --- a/src/text.cpp +++ b/src/text.cpp @@ -114,7 +114,7 @@ Text::~Text() textManager->removeText(this); if (--mInstances == 0) { - delete2(textManager); + delete2(textManager) for (int f = 0; f < 9; f ++) { if (mBubble.grid[f] != nullptr) diff --git a/src/unittests/gui/widgets/browserbox.cc b/src/unittests/gui/widgets/browserbox.cc index 2c0071313..a5e07a3ec 100644 --- a/src/unittests/gui/widgets/browserbox.cc +++ b/src/unittests/gui/widgets/browserbox.cc @@ -185,7 +185,7 @@ TEST_CASE("BrowserBox tests", "browserbox") delete Widget::getGloablFont(); Widget::setGlobalFont(nullptr); delete box; - delete2(client); + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/gui/widgets/staticbrowserbox.cc b/src/unittests/gui/widgets/staticbrowserbox.cc index 1dad81d84..41d4aab72 100644 --- a/src/unittests/gui/widgets/staticbrowserbox.cc +++ b/src/unittests/gui/widgets/staticbrowserbox.cc @@ -185,7 +185,7 @@ TEST_CASE("StaticBrowserBox tests", "browserbox") delete Widget::getGloablFont(); Widget::setGlobalFont(nullptr); delete box; - delete2(client); + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/gui/windowmanager.cc b/src/unittests/gui/windowmanager.cc index dbe7338a4..fa27e5dfd 100644 --- a/src/unittests/gui/windowmanager.cc +++ b/src/unittests/gui/windowmanager.cc @@ -218,7 +218,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV0(bankWindow, BankWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(bankWindow); + delete2(bankWindow) } SECTION("buyDialog1") { @@ -226,7 +226,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV0(dialog, BuyDialog); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("buyDialog2") { @@ -236,7 +236,7 @@ TEST_CASE("Windows tests", "windowmanager") DEFAULT_CURRENCY); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } #ifdef TMWA_SUPPORT SECTION("buyDialog3") @@ -247,7 +247,7 @@ TEST_CASE("Windows tests", "windowmanager") DEFAULT_CURRENCY); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } #endif // TMWA_SUPPORT SECTION("buyDialog4") @@ -266,9 +266,9 @@ TEST_CASE("Windows tests", "windowmanager") DEFAULT_CURRENCY); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(being); - delete2(map); + delete2(dialog) + delete2(being) + delete2(map) } SECTION("BuyingStoreSellDialog") { @@ -277,7 +277,7 @@ TEST_CASE("Windows tests", "windowmanager") BeingId_zero, 0); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("BuySellDialog1") { @@ -285,7 +285,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV(dialog, BuySellDialog, BeingId_zero); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("BuySellDialog2") { @@ -293,7 +293,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV(dialog, BuySellDialog, "user"); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("ChangeEmailDialog") { @@ -302,7 +302,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV(dialog, ChangeEmailDialog, data); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("ChangePasswordDialog") { @@ -311,7 +311,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV(dialog, ChangePasswordDialog, data); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("CharSelectDialog") { @@ -320,7 +320,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV(dialog, CharSelectDialog, data); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("CharCreateDialog") { @@ -332,8 +332,8 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV(dialog, CharCreateDialog, dialog2, 0); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(dialog2); + delete2(dialog) + delete2(dialog2) */ } SECTION("ChatWindow") @@ -342,14 +342,14 @@ TEST_CASE("Windows tests", "windowmanager") "Chat"); gui->draw(); mainGraphics->updateScreen(); - delete2(chatWindow); + delete2(chatWindow) } SECTION("ClanWindow") { CREATEWIDGETV0(clanWindow, ClanWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(clanWindow); + delete2(clanWindow) } SECTION("ConfirmDialog") { @@ -358,14 +358,14 @@ TEST_CASE("Windows tests", "windowmanager") "", "", SOUND_REQUEST, false, Modal_false, nullptr); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("CutinWindow") { CREATEWIDGETV0(cutInWindow, CutInWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(cutInWindow); + delete2(cutInWindow) } SECTION("DebugWindow") { @@ -373,14 +373,14 @@ TEST_CASE("Windows tests", "windowmanager") "Debug"); gui->draw(); mainGraphics->updateScreen(); - delete2(debugWindow); + delete2(debugWindow) } SECTION("didYouKnowWindow") { CREATEWIDGETV0(didYouKnowWindow, DidYouKnowWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(didYouKnowWindow); + delete2(didYouKnowWindow) } SECTION("EditDialog") { @@ -389,7 +389,7 @@ TEST_CASE("Windows tests", "windowmanager") "", "", "", 100, nullptr, Modal_false); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("EditServerDialog") { @@ -402,22 +402,22 @@ TEST_CASE("Windows tests", "windowmanager") serverDialog, mCurrentServer, 0); gui->draw(); mainGraphics->updateScreen(); - delete2(editServerDialog); - delete2(serverDialog); + delete2(editServerDialog) + delete2(serverDialog) } SECTION("EggSelectionDialog") { EggSelectionDialog *dialog = CREATEWIDGETR0(EggSelectionDialog); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("EmoteWindow") { EmoteWindow *dialog = CREATEWIDGETR0(EmoteWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("EquipmentWindow") { @@ -431,17 +431,17 @@ TEST_CASE("Windows tests", "windowmanager") equipment, being, false); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(being); - delete2(map); - delete2(equipment); + delete2(dialog) + delete2(being) + delete2(map) + delete2(equipment) } SECTION("helpWindow") { CREATEWIDGETV0(helpWindow, HelpWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(helpWindow); + delete2(helpWindow) } SECTION("InsertCardDialog") { @@ -459,8 +459,8 @@ TEST_CASE("Windows tests", "windowmanager") 0, item); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(item); + delete2(dialog) + delete2(item) } SECTION("InventoryWindow") { @@ -469,8 +469,8 @@ TEST_CASE("Windows tests", "windowmanager") inventory); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(inventory); + delete2(dialog) + delete2(inventory) } SECTION("ItemAmountWindow") { @@ -491,15 +491,15 @@ TEST_CASE("Windows tests", "windowmanager") 0); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(item); + delete2(dialog) + delete2(item) } SECTION("KillStats") { CREATEWIDGETV0(killStats, KillStats); gui->draw(); mainGraphics->updateScreen(); - delete2(killStats); + delete2(killStats) } SECTION("loginDialog") { @@ -510,14 +510,14 @@ TEST_CASE("Windows tests", "windowmanager") &settings.options.updateHost); gui->draw(); mainGraphics->updateScreen(); - delete2(loginDialog); + delete2(loginDialog) } SECTION("MailEditWindow") { CREATEWIDGETV0(mailEditWindow, MailEditWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(mailEditWindow); + delete2(mailEditWindow) } SECTION("MailViewWindow") { @@ -527,7 +527,7 @@ TEST_CASE("Windows tests", "windowmanager") 1); gui->draw(); mainGraphics->updateScreen(); - delete2(mailViewWindow); + delete2(mailViewWindow) delete message; } SECTION("MailWindow") @@ -535,35 +535,35 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV0(mailWindow, MailWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(mailWindow); + delete2(mailWindow) } SECTION("Minimap") { CREATEWIDGETV0(minimap, Minimap); gui->draw(); mainGraphics->updateScreen(); - delete2(minimap); + delete2(minimap) } SECTION("MiniStatusWindow") { CREATEWIDGETV0(miniStatusWindow, MiniStatusWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(miniStatusWindow); + delete2(miniStatusWindow) } SECTION("NpcDialog") { NpcDialog *dialog = CREATEWIDGETR(NpcDialog, BeingId_zero); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("NpcSellDialog") { NpcSellDialog *dialog = CREATEWIDGETR(NpcSellDialog, BeingId_zero); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("OkDialog") { @@ -572,21 +572,21 @@ TEST_CASE("Windows tests", "windowmanager") ShowCenter_true, nullptr, 100); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("OutfitWindow") { CREATEWIDGETV0(outfitWindow, OutfitWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(outfitWindow); + delete2(outfitWindow) } SECTION("QuestsWindow") { CREATEWIDGETV0(questsWindow, QuestsWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(questsWindow); + delete2(questsWindow) } SECTION("QuitDialog") { @@ -594,7 +594,7 @@ TEST_CASE("Windows tests", "windowmanager") &quitDialog); gui->draw(); mainGraphics->updateScreen(); - delete2(quitDialog); + delete2(quitDialog) } SECTION("RegisterDialog") { @@ -602,7 +602,7 @@ TEST_CASE("Windows tests", "windowmanager") loginData); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("serversDialog") { @@ -613,7 +613,7 @@ TEST_CASE("Windows tests", "windowmanager") settings.configDir); gui->draw(); mainGraphics->updateScreen(); - delete2(serverDialog); + delete2(serverDialog) } SECTION("serversInfoWindow") { @@ -622,14 +622,14 @@ TEST_CASE("Windows tests", "windowmanager") mCurrentServer); gui->draw(); mainGraphics->updateScreen(); - delete2(serverInfoWindow); + delete2(serverInfoWindow) } SECTION("setupWindow") { CREATEWIDGETV0(setupWindow, SetupWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(setupWindow); + delete2(setupWindow) } SECTION("ShopSellDialog") { @@ -640,7 +640,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV0(shopWindow, ShopWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(shopWindow); + delete2(shopWindow) } SECTION("ShortcutWindow1") { @@ -654,7 +654,7 @@ TEST_CASE("Windows tests", "windowmanager") 0); gui->draw(); mainGraphics->updateScreen(); - delete2(itemShortcutWindow); + delete2(itemShortcutWindow) } SECTION("ShortcutWindow2") { @@ -665,21 +665,21 @@ TEST_CASE("Windows tests", "windowmanager") 0); gui->draw(); mainGraphics->updateScreen(); - delete2(itemShortcutWindow); + delete2(itemShortcutWindow) } SECTION("SkillDialog") { CREATEWIDGETV0(skillDialog, SkillDialog); gui->draw(); mainGraphics->updateScreen(); - delete2(skillDialog); + delete2(skillDialog) } SECTION("SocialWindow") { CREATEWIDGETV0(socialWindow, SocialWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(socialWindow); + delete2(socialWindow) } SECTION("StatusWindow") { @@ -690,7 +690,7 @@ TEST_CASE("Windows tests", "windowmanager") CREATEWIDGETV0(statusWindow, StatusWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(statusWindow); + delete2(statusWindow) delete groups[0]; groups.clear(); } @@ -702,8 +702,8 @@ TEST_CASE("Windows tests", "windowmanager") textCommand); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); - delete2(textCommand); + delete2(dialog) + delete2(textCommand) } SECTION("TextDialog") @@ -712,7 +712,7 @@ TEST_CASE("Windows tests", "windowmanager") "", "", nullptr, false); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("TextSelectDialog") { @@ -720,14 +720,14 @@ TEST_CASE("Windows tests", "windowmanager") "", "", AllowQuit_false); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("TradeWindow") { CREATEWIDGETV0(tradeWindow, TradeWindow); gui->draw(); mainGraphics->updateScreen(); - delete2(tradeWindow); + delete2(tradeWindow) } SECTION("UpdaterWindow") { @@ -735,14 +735,14 @@ TEST_CASE("Windows tests", "windowmanager") "", "", false, UpdateType::Skip); gui->draw(); mainGraphics->updateScreen(); - delete2(updaterWindow); + delete2(updaterWindow) } SECTION("WhoIsOnline") { CREATEWIDGETV0(whoIsOnline, WhoIsOnline); gui->draw(); mainGraphics->updateScreen(); - delete2(whoIsOnline); + delete2(whoIsOnline) } SECTION("WorldSelectDialog") { @@ -751,77 +751,77 @@ TEST_CASE("Windows tests", "windowmanager") worlds); gui->draw(); mainGraphics->updateScreen(); - delete2(dialog); + delete2(dialog) } SECTION("popupMenu") { CREATEWIDGETV0(popupMenu, PopupMenu); gui->draw(); mainGraphics->updateScreen(); - delete2(popupMenu); + delete2(popupMenu) } SECTION("skillPopup") { CREATEWIDGETV0(skillPopup, SkillPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(skillPopup); + delete2(skillPopup) } SECTION("SpeechBubble") { SpeechBubble *bubble = CREATEWIDGETR0(SpeechBubble); gui->draw(); mainGraphics->updateScreen(); - delete2(bubble); + delete2(bubble) } SECTION("beingPopup") { CREATEWIDGETV0(beingPopup, BeingPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(beingPopup); + delete2(beingPopup) } SECTION("textPopup") { CREATEWIDGETV0(textPopup, TextPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(textPopup); + delete2(textPopup) } SECTION("textBoxPopup") { CREATEWIDGETV0(textBoxPopup, TextBoxPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(textBoxPopup); + delete2(textBoxPopup) } SECTION("itemPopup") { CREATEWIDGETV0(itemPopup, ItemPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(itemPopup); + delete2(itemPopup) } SECTION("spellPopup") { CREATEWIDGETV0(spellPopup, SpellPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(spellPopup); + delete2(spellPopup) } SECTION("StatusPopup") { StatusPopup *status = CREATEWIDGETR0(StatusPopup); gui->draw(); mainGraphics->updateScreen(); - delete2(status); + delete2(status) } SECTION("desktop") { CREATEWIDGETV(desktop, Desktop, nullptr); gui->draw(); mainGraphics->updateScreen(); - delete2(desktop); + delete2(desktop) } SECTION("connectionDialog") { @@ -831,18 +831,18 @@ TEST_CASE("Windows tests", "windowmanager") State::SWITCH_SERVER); gui->draw(); mainGraphics->updateScreen(); - delete2(connectionDialog); + delete2(connectionDialog) } UnitsDb::unload(); - delete2(localPlayer); - delete2(userPalette); - delete2(client); - delete2(serverFeatures); - delete2(inventoryHandler); - delete2(charServerHandler); - delete2(playerHandler); - delete2(gui); + delete2(localPlayer) + delete2(userPalette) + delete2(client) + delete2(serverFeatures) + delete2(inventoryHandler) + delete2(charServerHandler) + delete2(playerHandler) + delete2(gui) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); @@ -998,13 +998,13 @@ TEST_CASE("WindowManager", "create windows") settings.limitFps = true; UnitsDb::unload(); - delete2(localPlayer); - delete2(client); - delete2(serverFeatures); - delete2(inventoryHandler); - delete2(charServerHandler); - delete2(playerHandler); - delete2(gui); + delete2(localPlayer) + delete2(client) + delete2(serverFeatures) + delete2(inventoryHandler) + delete2(charServerHandler) + delete2(playerHandler) + delete2(gui) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); diff --git a/src/unittests/integrity.cc b/src/unittests/integrity.cc index b52d991a0..8b07d4732 100644 --- a/src/unittests/integrity.cc +++ b/src/unittests/integrity.cc @@ -352,8 +352,8 @@ TEST_CASE("integrity tests", "integrity") ResourceManager::cleanOrphans(true); - delete2(userPalette); - delete2(client); + delete2(userPalette) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); diff --git a/src/unittests/resources/dye/dye.cc b/src/unittests/resources/dye/dye.cc index bf9ea5cb4..68f473378 100644 --- a/src/unittests/resources/dye/dye.cc +++ b/src/unittests/resources/dye/dye.cc @@ -2004,9 +2004,9 @@ TEST_CASE("Dye real dye", "") dyeCheck("|A:#0000FFFF,FF000050", "arrow_up_A.png"); } - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); VirtFs::unmountDirSilent("data/test"); diff --git a/src/unittests/resources/dye/dyepalette.cc b/src/unittests/resources/dye/dyepalette.cc index e5da0be04..e31052bc5 100644 --- a/src/unittests/resources/dye/dyepalette.cc +++ b/src/unittests/resources/dye/dyepalette.cc @@ -392,7 +392,7 @@ TEST_CASE("DyePalette tests", "") REQUIRE(palette.mColors[0].value[2] == 46); REQUIRE(palette.mColors[0].value[3] == 0x77); } - delete2(client); + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); VirtFs::unmountDirSilent("data/test"); diff --git a/src/unittests/resources/map/maplayer/drawfringenormal.cc b/src/unittests/resources/map/maplayer/drawfringenormal.cc index 963084f8b..c94429a0f 100644 --- a/src/unittests/resources/map/maplayer/drawfringenormal.cc +++ b/src/unittests/resources/map/maplayer/drawfringenormal.cc @@ -514,13 +514,13 @@ TEST_CASE("MapLayer drawFringe normal", "") REQUIRE(mock->mDraws[7].image == img1); } - delete2(localPlayer); + delete2(localPlayer) delete map; delete img1; delete img2; delete img3; delete mock; - delete2(theme); + delete2(theme) GraphicsManager::deleteRenderers(); ResourceManager::cleanOrphans(true); ResourceManager::deleteInstance(); diff --git a/src/unittests/resources/map/maplayer/drawfringesimple.cc b/src/unittests/resources/map/maplayer/drawfringesimple.cc index c1d1e858a..6fded0f43 100644 --- a/src/unittests/resources/map/maplayer/drawfringesimple.cc +++ b/src/unittests/resources/map/maplayer/drawfringesimple.cc @@ -492,13 +492,13 @@ TEST_CASE("MapLayer drawFringe simple", "") REQUIRE(mock->mDraws[1].image == img2); } - delete2(localPlayer); + delete2(localPlayer) delete map; delete img1; delete img2; delete img3; delete mock; - delete2(theme); + delete2(theme) GraphicsManager::deleteRenderers(); ResourceManager::cleanOrphans(true); ResourceManager::deleteInstance(); diff --git a/src/unittests/resources/map/speciallayer.cc b/src/unittests/resources/map/speciallayer.cc index 17f7e18b9..892d20737 100644 --- a/src/unittests/resources/map/speciallayer.cc +++ b/src/unittests/resources/map/speciallayer.cc @@ -280,9 +280,9 @@ TEST_CASE("SpecialLayer updateCache", "") delete layer; ResourceManager::cleanOrphans(false); - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/resources/resourcemanager/resourcemanager.cc b/src/unittests/resources/resourcemanager/resourcemanager.cc index d40b197d1..6bb03ecb5 100644 --- a/src/unittests/resources/resourcemanager/resourcemanager.cc +++ b/src/unittests/resources/resourcemanager/resourcemanager.cc @@ -685,9 +685,9 @@ TEST_CASE("resourcemanager", "resourcemanager") REQUIRE(ResourceManager::getDeletedResources().empty() == true); } - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/resources/sdlimagehelper.cc b/src/unittests/resources/sdlimagehelper.cc index cb20801fc..dad9ddc0e 100644 --- a/src/unittests/resources/sdlimagehelper.cc +++ b/src/unittests/resources/sdlimagehelper.cc @@ -548,9 +548,9 @@ TEST_CASE("sdlimagehelper combineSurface", "") MSDL_FreeSurface(surface2); } - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/resources/sprite/animatedsprite.cc b/src/unittests/resources/sprite/animatedsprite.cc index 33b5529b2..376dfb79d 100644 --- a/src/unittests/resources/sprite/animatedsprite.cc +++ b/src/unittests/resources/sprite/animatedsprite.cc @@ -192,7 +192,7 @@ TEST_CASE("AnimatedSprite tests", "animatedsprite") delete sprite2; } - delete2(client); + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/sdl.cc b/src/unittests/sdl.cc index 54b415e66..c826fdfec 100644 --- a/src/unittests/sdl.cc +++ b/src/unittests/sdl.cc @@ -396,9 +396,9 @@ TEST_CASE("sdl tests", "sdl") } ResourceManager::cleanOrphans(true); - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); diff --git a/src/unittests/utils/chatutils.cc b/src/unittests/utils/chatutils.cc index 1069f6b5d..0121094d8 100644 --- a/src/unittests/utils/chatutils.cc +++ b/src/unittests/utils/chatutils.cc @@ -249,9 +249,9 @@ TEST_CASE("chatutils replaceVars", "") Party::clearParties(); } - delete2(actorManager); - delete2(localPlayer); - delete2(client); + delete2(actorManager) + delete2(localPlayer) + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/utils/checkutils.cc b/src/unittests/utils/checkutils.cc index a80a24c74..8244248ca 100644 --- a/src/unittests/utils/checkutils.cc +++ b/src/unittests/utils/checkutils.cc @@ -76,7 +76,7 @@ static bool testFailAlways1() static bool testFailAlways2() { - reportAlways("test fail"); + reportAlways("test fail") return false; } @@ -96,7 +96,7 @@ TEST_CASE("CheckUtils", "") SECTION("reportAlways") { - reportAlwaysReal("test report"); + reportAlwaysReal("test report") } SECTION("failFalse") diff --git a/src/unittests/utils/itemxmlutils.cc b/src/unittests/utils/itemxmlutils.cc index 82cb5e25f..a73225f85 100644 --- a/src/unittests/utils/itemxmlutils.cc +++ b/src/unittests/utils/itemxmlutils.cc @@ -211,9 +211,9 @@ TEST_CASE("itemxmlutils readItemStatsString 1", "") { delete (*it).second; } - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data/test"); VirtFs::unmountDirSilent("../data/test"); @@ -413,9 +413,9 @@ TEST_CASE("itemxmlutils readItemStatsVector 1", "") { delete (*it).second; } - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data/test"); VirtFs::unmountDirSilent("../data/test"); diff --git a/src/unittests/utils/translation/poparser.cc b/src/unittests/utils/translation/poparser.cc index aede83335..233eaa2a9 100644 --- a/src/unittests/utils/translation/poparser.cc +++ b/src/unittests/utils/translation/poparser.cc @@ -130,7 +130,7 @@ TEST_CASE("PoParser tests", "PoParser") delete parser; delete dict; } - delete2(client); + delete2(client) VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); } diff --git a/src/unittests/utils/xml.cc b/src/unittests/utils/xml.cc index 4711873ce..8165c23ef 100644 --- a/src/unittests/utils/xml.cc +++ b/src/unittests/utils/xml.cc @@ -395,8 +395,8 @@ TEST_CASE("xml doc", "") ::remove(tempXmlName); } - delete2(theme); - delete2(client); + delete2(theme) + delete2(client) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); diff --git a/src/unittests/utils/xmlutils.cc b/src/unittests/utils/xmlutils.cc index 5d1bc4c46..0cc23fd90 100644 --- a/src/unittests/utils/xmlutils.cc +++ b/src/unittests/utils/xmlutils.cc @@ -96,9 +96,9 @@ TEST_CASE("xmlutils readXmlIntVector 1", "") REQUIRE(1 == arr[3]); REQUIRE(1 == arr[4]); - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); @@ -154,9 +154,9 @@ TEST_CASE("xmlutils readXmlStringMap 1", "") REQUIRE(arr["xD"] == "1"); REQUIRE(arr["Metal"] == "26"); - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data"); VirtFs::unmountDirSilent("../data"); @@ -214,9 +214,9 @@ TEST_CASE("xmlutils readXmlIntMap 1", "") REQUIRE(arr[10] == 20); REQUIRE(arr[3] == 0); - delete2(userPalette); - delete2(theme); - delete2(client); + delete2(userPalette) + delete2(theme) + delete2(client) ResourceManager::deleteInstance(); VirtFs::unmountDirSilent("data/test"); VirtFs::unmountDirSilent("../data/test"); diff --git a/src/utils/copynpaste.cpp b/src/utils/copynpaste.cpp index 3b2b4ae8a..942801de2 100644 --- a/src/utils/copynpaste.cpp +++ b/src/utils/copynpaste.cpp @@ -387,7 +387,7 @@ bool retrieveBuffer(std::string& text, size_t& pos) { SDL_SysWMinfo info; - SDL_VERSION(&info.version); + SDL_VERSION(&info.version) if (SDL::getWindowWMInfo(mainGraphics->getWindow(), &info)) { Display *const dpy = info.info.x11.display; diff --git a/src/utils/dumpsizes.cpp b/src/utils/dumpsizes.cpp index f1a9a613a..8275133a5 100644 --- a/src/utils/dumpsizes.cpp +++ b/src/utils/dumpsizes.cpp @@ -27,7 +27,7 @@ #include "debug.h" #define dumpSize(str) \ - logger->log(" sizeof("#str"): %d", CAST_S32(sizeof(str))); + logger->log(" sizeof("#str"): %d", CAST_S32(sizeof(str))) void dumpSizes() { diff --git a/src/utils/itemxmlutils.cpp b/src/utils/itemxmlutils.cpp index 8ecbeeda6..ac8e48e2b 100644 --- a/src/utils/itemxmlutils.cpp +++ b/src/utils/itemxmlutils.cpp @@ -81,7 +81,7 @@ void readItemStatsString(std::string &effect, { if (translator == nullptr) { - reportAlways("called readItemStatsString without translator"); + reportAlways("called readItemStatsString without translator") return; } @@ -102,7 +102,7 @@ void readItemStatsVector(STD_VECTOR<std::string> &effect, { if (translator == nullptr) { - reportAlways("called readItemStatsVector without translator"); + reportAlways("called readItemStatsVector without translator") return; } diff --git a/src/utils/pnglib.cpp b/src/utils/pnglib.cpp index 4393be3e2..6d591515d 100644 --- a/src/utils/pnglib.cpp +++ b/src/utils/pnglib.cpp @@ -47,7 +47,7 @@ bool PngLib::writePNG(SDL_Surface *const surface, nullptr, nullptr, nullptr); if (png_ptr == nullptr) { - reportAlways("Had trouble creating png_structp"); + reportAlways("Had trouble creating png_structp") return false; } @@ -55,14 +55,14 @@ bool PngLib::writePNG(SDL_Surface *const surface, if (info_ptr == nullptr) { png_destroy_write_struct(&png_ptr, static_cast<png_infopp>(nullptr)); - reportAlways("Could not create png_info"); + reportAlways("Could not create png_info") return false; } if (setjmp(png_jmpbuf(png_ptr))) { png_destroy_write_struct(&png_ptr, &info_ptr); - reportAlways("problem writing to %s", filename.c_str()); + reportAlways("problem writing to %s", filename.c_str()) return false; } @@ -70,7 +70,7 @@ bool PngLib::writePNG(SDL_Surface *const surface, if (fp == nullptr) { reportAlways("could not open file %s for writing", - filename.c_str()); + filename.c_str()) return false; } diff --git a/src/utils/sdl2logger.cpp b/src/utils/sdl2logger.cpp index 16d8aa7d0..8bba68d6d 100644 --- a/src/utils/sdl2logger.cpp +++ b/src/utils/sdl2logger.cpp @@ -117,7 +117,7 @@ static void logCallback(void *userData A_UNUSED, } reportAlways("%s %s", message.c_str(), - msg); + msg) } static SDL_AssertState assertCallback(const SDL_AssertData *data, @@ -128,7 +128,7 @@ static SDL_AssertState assertCallback(const SDL_AssertData *data, data->function, data->filename, data->linenum, - data->condition); + data->condition) return SDL_ASSERTION_IGNORE; } diff --git a/src/utils/sdlhelper.cpp b/src/utils/sdlhelper.cpp index 474840d2a..7e10352f4 100644 --- a/src/utils/sdlhelper.cpp +++ b/src/utils/sdlhelper.cpp @@ -128,7 +128,7 @@ void *SDL::createGLContext(SDL_Surface *const window A_UNUSED, const int profile) { SDL_SysWMinfo info; - SDL_VERSION(&info.version); + SDL_VERSION(&info.version) SDL_GetWMInfo(&info); void *context = GlxHelper::createContext(info.info.x11.window, info.info.x11.display, major, minor, profile); @@ -162,7 +162,7 @@ void *SDL::createGLContext(SDL_Surface *const window A_UNUSED, void SDL::makeCurrentContext(void *const context) { SDL_SysWMinfo info; - SDL_VERSION(&info.version); + SDL_VERSION(&info.version) SDL_GetWMInfo(&info); GlxHelper::makeCurrent(info.info.x11.window, info.info.x11.display, diff --git a/src/utils/translation/translationmanager.cpp b/src/utils/translation/translationmanager.cpp index 5b1c2d67a..0f691f5f8 100644 --- a/src/utils/translation/translationmanager.cpp +++ b/src/utils/translation/translationmanager.cpp @@ -66,9 +66,9 @@ void TranslationManager::loadGettextLang() void TranslationManager::close() { - delete2(translator); - delete2(dictionary); - delete2(reverseDictionary); + delete2(translator) + delete2(dictionary) + delete2(reverseDictionary) } PoDict *TranslationManager::loadLang(const LangVect &lang, diff --git a/src/utils/xml/libxml.cpp b/src/utils/xml/libxml.cpp index 7fcdfd7fc..a4c1ea7ba 100644 --- a/src/utils/xml/libxml.cpp +++ b/src/utils/xml/libxml.cpp @@ -118,7 +118,7 @@ namespace XML if (size < 0) { reportAlways("Error loading XML file %s", - filename.c_str()); + filename.c_str()) } else { @@ -131,7 +131,7 @@ namespace XML else if (skipError == SkipError_false) { reportAlways("Error loading XML file %s", - filename.c_str()); + filename.c_str()) } } @@ -142,12 +142,12 @@ namespace XML if (mDoc == nullptr) { - reportAlways("Error parsing XML file %s", filename.c_str()); + reportAlways("Error parsing XML file %s", filename.c_str()) } } else if (skipError == SkipError_false) { - reportAlways("Error loading XML file %s", filename.c_str()); + reportAlways("Error loading XML file %s", filename.c_str()) } mIsValid = valid; BLOCK_END("XML::Document::Document") @@ -304,7 +304,7 @@ namespace XML void initXML() { xmlInitParser(); - LIBXML_TEST_VERSION; + LIBXML_TEST_VERSION // Suppress libxml2 error messages xmlSetGenericErrorFunc(nullptr, &xmlErrorLogger); diff --git a/src/utils/xml/pugixml.cpp b/src/utils/xml/pugixml.cpp index 4919b34a8..dfc40a970 100644 --- a/src/utils/xml/pugixml.cpp +++ b/src/utils/xml/pugixml.cpp @@ -96,7 +96,7 @@ namespace XML if (size < 0) { reportAlways("Error loading XML file %s", - filename.c_str()); + filename.c_str()) } else { @@ -109,7 +109,7 @@ namespace XML else if (skipError == SkipError_false) { reportAlways("Error loading XML file %s", - filename.c_str()); + filename.c_str()) } } @@ -135,7 +135,7 @@ namespace XML } else if (skipError == SkipError_false) { - reportAlways("Error loading %s", filename.c_str()); + reportAlways("Error loading %s", filename.c_str()) } mIsValid = valid; BLOCK_END("XML::Document::Document") diff --git a/src/utils/xml/tinyxml2.cpp b/src/utils/xml/tinyxml2.cpp index 48ea9c891..99fa09001 100644 --- a/src/utils/xml/tinyxml2.cpp +++ b/src/utils/xml/tinyxml2.cpp @@ -95,7 +95,7 @@ namespace XML if (size < 0) { reportAlways("Error loading XML file %s", - filename.c_str()); + filename.c_str()) } else { @@ -108,7 +108,7 @@ namespace XML else if (skipError == SkipError_false) { reportAlways("Error loading XML file %s", - filename.c_str()); + filename.c_str()) } } @@ -128,7 +128,7 @@ namespace XML } else if (skipError == SkipError_false) { - reportAlways("Error loading %s", filename.c_str()); + reportAlways("Error loading %s", filename.c_str()) } mIsValid = valid; BLOCK_END("XML::Document::Document") |