summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--src/actormanager.cpp6
-rw-r--r--src/being/playerrelations.cpp2
-rw-r--r--src/gui/fonts/textchunk.cpp4
-rw-r--r--src/gui/fonts/textchunksmall.cpp5
-rw-r--r--src/gui/mouseoverlink.h4
-rw-r--r--src/gui/viewport.cpp4
-rw-r--r--src/gui/widgets/button.cpp2
-rw-r--r--src/gui/widgets/itemcontainer.cpp8
-rw-r--r--src/gui/widgets/widget.cpp2
-rw-r--r--src/gui/windows/inventorywindow.cpp2
-rw-r--r--src/gui/windows/tradewindow.cpp2
-rw-r--r--src/input/inputmanager.cpp4
-rw-r--r--src/input/joystick.cpp2
-rw-r--r--src/input/key.cpp11
-rw-r--r--src/net/serverinfo.h10
-rw-r--r--src/render/sdlgraphics.cpp2
-rw-r--r--src/resources/map/speciallayer.cpp2
-rw-r--r--src/resources/sdlgfxblitfunc.cpp2
-rw-r--r--src/resources/sprite/animatedsprite.cpp4
-rw-r--r--src/resources/wallpaper.cpp3
-rw-r--r--src/utils/stringutils.cpp5
21 files changed, 47 insertions, 39 deletions
diff --git a/src/actormanager.cpp b/src/actormanager.cpp
index 2649cb230..fd24147e9 100644
--- a/src/actormanager.cpp
+++ b/src/actormanager.cpp
@@ -95,11 +95,11 @@ class FindBeingFunctor final
const Vector &pos = b->getPixelPositionF();
// +++ probably here need use int positions and not float?
// but for now correct int positions only in Being
- return (CAST_U32(pos.x) / mapTileSize == x &&
+ return CAST_U32(pos.x) / mapTileSize == x &&
(CAST_U32(pos.y) / mapTileSize == y
|| CAST_U32(pos.y) / mapTileSize == other_y) &&
b->isAlive() && (type == ActorType::Unknown
- || b->getType() == type));
+ || b->getType() == type);
}
uint16_t x, y;
@@ -189,7 +189,7 @@ class SortBeingFunctor final
return w1 < w2;
}
- return (being1->getName() < being2->getName());
+ return being1->getName() < being2->getName();
}
StringIntMap *attackBeings;
StringIntMap *priorityBeings;
diff --git a/src/being/playerrelations.cpp b/src/being/playerrelations.cpp
index a1e9a457c..fe774432b 100644
--- a/src/being/playerrelations.cpp
+++ b/src/being/playerrelations.cpp
@@ -613,7 +613,7 @@ bool PlayerRelationsManager::isGoodName(Being *const being) const
if (being == nullptr)
return false;
if (being->getGoodStatus() != -1)
- return (being->getGoodStatus() == 1);
+ return being->getGoodStatus() == 1;
const std::string &name = being->getName();
const size_t size = name.size();
diff --git a/src/gui/fonts/textchunk.cpp b/src/gui/fonts/textchunk.cpp
index 694790881..22dcd18dc 100644
--- a/src/gui/fonts/textchunk.cpp
+++ b/src/gui/fonts/textchunk.cpp
@@ -89,8 +89,8 @@ TextChunk::~TextChunk()
bool TextChunk::operator==(const TextChunk &restrict chunk) const
{
- return (chunk.text == text && chunk.color == color
- && chunk.color2 == color2);
+ return chunk.text == text && chunk.color == color
+ && chunk.color2 == color2;
}
void TextChunk::generate(TTF_Font *restrict const font,
diff --git a/src/gui/fonts/textchunksmall.cpp b/src/gui/fonts/textchunksmall.cpp
index c7fbf6051..73e8ad386 100644
--- a/src/gui/fonts/textchunksmall.cpp
+++ b/src/gui/fonts/textchunksmall.cpp
@@ -50,8 +50,9 @@ TextChunkSmall &TextChunkSmall::operator=(const TextChunkSmall &chunk)
bool TextChunkSmall::operator==(const TextChunkSmall &chunk) const
{
- return (chunk.text == text && chunk.color == color
- && chunk.color2 == color2);
+ return chunk.text == text &&
+ chunk.color == color &&
+ chunk.color2 == color2;
}
bool TextChunkSmall::operator<(const TextChunkSmall &chunk) const
diff --git a/src/gui/mouseoverlink.h b/src/gui/mouseoverlink.h
index a26d38d6b..758d02b6e 100644
--- a/src/gui/mouseoverlink.h
+++ b/src/gui/mouseoverlink.h
@@ -39,10 +39,10 @@ struct MouseOverLink final
bool operator() (const BrowserLink &link) const
{
- return (mX >= link.x1 &&
+ return mX >= link.x1 &&
mX < link.x2 &&
mY >= link.y1 &&
- mY < link.y2);
+ mY < link.y2;
}
const int mX;
const int mY;
diff --git a/src/gui/viewport.cpp b/src/gui/viewport.cpp
index 168ec317a..5715eae3a 100644
--- a/src/gui/viewport.cpp
+++ b/src/gui/viewport.cpp
@@ -467,8 +467,8 @@ bool Viewport::leftMouseAction()
case ActorType::SkillUnit:
if (!stopAttack)
{
- if ((localPlayer->withinAttackRange(mHoverBeing) ||
- inputManager.isActionActive(InputAction::ATTACK)))
+ if (localPlayer->withinAttackRange(mHoverBeing) ||
+ inputManager.isActionActive(InputAction::ATTACK))
{
validateSpeed();
if (!mStatsReUpdated && localPlayer != mHoverBeing)
diff --git a/src/gui/widgets/button.cpp b/src/gui/widgets/button.cpp
index df46014a3..d09a6b3cb 100644
--- a/src/gui/widgets/button.cpp
+++ b/src/gui/widgets/button.cpp
@@ -873,7 +873,7 @@ void Button::keyReleased(KeyEvent& event)
bool Button::isPressed2() const
{
- return (mPressed || isPressed());
+ return mPressed || isPressed();
}
bool Button::isPressed() const
diff --git a/src/gui/widgets/itemcontainer.cpp b/src/gui/widgets/itemcontainer.cpp
index 3d5c3829f..8dd0f1210 100644
--- a/src/gui/widgets/itemcontainer.cpp
+++ b/src/gui/widgets/itemcontainer.cpp
@@ -101,7 +101,7 @@ namespace
return item1->getInvIndex() <
item2->getInvIndex();
}
- return (name1 < name2);
+ return name1 < name2;
}
} itemAlphaInvSorter;
@@ -151,7 +151,7 @@ namespace
return pair1->mItem->getInvIndex() <
pair2->mItem->getInvIndex();
}
- return (name1 < name2);
+ return name1 < name2;
}
return w1 < w2;
}
@@ -181,7 +181,7 @@ namespace
return pair1->mItem->getInvIndex() <
pair2->mItem->getInvIndex();
}
- return (name1 < name2);
+ return name1 < name2;
}
return c1 < c2;
}
@@ -211,7 +211,7 @@ namespace
return pair1->mItem->getInvIndex() <
pair2->mItem->getInvIndex();
}
- return (name1 < name2);
+ return name1 < name2;
}
return t1 < t2;
}
diff --git a/src/gui/widgets/widget.cpp b/src/gui/widgets/widget.cpp
index 8b49fddfd..47d2e0f5b 100644
--- a/src/gui/widgets/widget.cpp
+++ b/src/gui/widgets/widget.cpp
@@ -184,7 +184,7 @@ bool Widget::isFocused() const
if (mFocusHandler == nullptr)
return false;
- return (mFocusHandler->isFocused(this));
+ return mFocusHandler->isFocused(this);
}
void Widget::setFocusable(const bool focusable)
diff --git a/src/gui/windows/inventorywindow.cpp b/src/gui/windows/inventorywindow.cpp
index 27caa2019..cd688be9b 100644
--- a/src/gui/windows/inventorywindow.cpp
+++ b/src/gui/windows/inventorywindow.cpp
@@ -558,7 +558,7 @@ void InventoryWindow::mouseClicked(MouseEvent &event)
const bool mod = (isStorageActive() &&
inputManager.isActionActive(InputAction::STOP_ATTACK));
- const bool mod2 = ((tradeWindow != nullptr) &&
+ const bool mod2 = (tradeWindow != nullptr &&
tradeWindow->isWindowVisible() &&
inputManager.isActionActive(InputAction::STOP_ATTACK));
diff --git a/src/gui/windows/tradewindow.cpp b/src/gui/windows/tradewindow.cpp
index 143a625e7..66f4e0550 100644
--- a/src/gui/windows/tradewindow.cpp
+++ b/src/gui/windows/tradewindow.cpp
@@ -549,5 +549,5 @@ bool TradeWindow::checkItem(const Item *const item) const
bool TradeWindow::isInpupFocused() const
{
- return ((mMoneyField != nullptr) && mMoneyField->isFocused());
+ return mMoneyField != nullptr && mMoneyField->isFocused();
}
diff --git a/src/input/inputmanager.cpp b/src/input/inputmanager.cpp
index 814e2ac76..b0aa18da5 100644
--- a/src/input/inputmanager.cpp
+++ b/src/input/inputmanager.cpp
@@ -816,8 +816,8 @@ bool InputManager::checkKey(const InputActionData *restrict const key) const
if ((key == nullptr) || (key->condition & mMask) != key->condition)
return false;
- return (key->modKeyIndex == InputAction::NO_VALUE
- || isActionActive0(key->modKeyIndex));
+ return key->modKeyIndex == InputAction::NO_VALUE
+ || isActionActive0(key->modKeyIndex);
}
bool InputManager::invokeKey(const InputActionData *restrict const key,
diff --git a/src/input/joystick.cpp b/src/input/joystick.cpp
index 0c859f50d..e01177059 100644
--- a/src/input/joystick.cpp
+++ b/src/input/joystick.cpp
@@ -349,7 +349,7 @@ bool Joystick::validate() const
if (mCalibrating || !mEnabled || !mCalibrated)
return false;
- return (mUseInactive || settings.inputFocused);
+ return mUseInactive || settings.inputFocused;
}
void Joystick::handleRepeat(const int time)
diff --git a/src/input/key.cpp b/src/input/key.cpp
index 97019e67e..13646dc8e 100644
--- a/src/input/key.cpp
+++ b/src/input/key.cpp
@@ -88,10 +88,11 @@ bool Key::isNumber() const
bool Key::isLetter() const
{
- return (((mValue >= 65 && mValue <= 90)
- || (mValue >= 97 && mValue <= 122)
- || (mValue >= 192 && mValue <= 255))
- && (mValue != 215) && (mValue != 247));
+ return ((mValue >= 65 && mValue <= 90) ||
+ (mValue >= 97 && mValue <= 122) ||
+ (mValue >= 192 && mValue <= 255)) &&
+ mValue != 215 &&
+ mValue != 247;
}
int Key::getValue() const
@@ -106,5 +107,5 @@ bool Key::operator==(const Key& key) const
bool Key::operator!=(const Key& key) const
{
- return (mValue != key.mValue);
+ return mValue != key.mValue;
}
diff --git a/src/net/serverinfo.h b/src/net/serverinfo.h
index f5da554ee..2bdec0251 100644
--- a/src/net/serverinfo.h
+++ b/src/net/serverinfo.h
@@ -181,14 +181,16 @@ class ServerInfo final
bool operator==(const ServerInfo &other) const
{
- return (name == other.name && hostname == other.hostname &&
- port == other.port);
+ return name == other.name &&
+ hostname == other.hostname &&
+ port == other.port;
}
bool operator!=(const ServerInfo &other) const
{
- return (name != other.name || hostname != other.hostname ||
- port != other.port);
+ return name != other.name ||
+ hostname != other.hostname ||
+ port != other.port;
}
static ServerTypeT parseType(const std::string &serverType)
diff --git a/src/render/sdlgraphics.cpp b/src/render/sdlgraphics.cpp
index bf4e1a258..9220f530e 100644
--- a/src/render/sdlgraphics.cpp
+++ b/src/render/sdlgraphics.cpp
@@ -1054,7 +1054,7 @@ void SDLGraphics::fillRectangle(const Rect &restrict rectangle) restrict2
const unsigned pb = (pixel & bMask) * mColor.a;
const unsigned pg = (pixel & gMask) * mColor.a;
const unsigned pr = (pixel & rMask) * mColor.a;
- const unsigned a0 = (255 - mColor.a);
+ const unsigned a0 = 255 - mColor.a;
const unsigned int a1 = a0 * bShift;
const unsigned int a2 = a0 * gShift;
diff --git a/src/resources/map/speciallayer.cpp b/src/resources/map/speciallayer.cpp
index 27713e074..ad186c2d9 100644
--- a/src/resources/map/speciallayer.cpp
+++ b/src/resources/map/speciallayer.cpp
@@ -104,7 +104,7 @@ void SpecialLayer::addRoad(const Path &road)
{
FOR_EACH (Path::const_iterator, i, road)
{
- const Position &pos = (*i);
+ const Position &pos = *i;
MapItem *const item = getTile(pos.x, pos.y);
if (item == nullptr)
setTile(pos.x, pos.y, new MapItem(MapItemType::ROAD));
diff --git a/src/resources/sdlgfxblitfunc.cpp b/src/resources/sdlgfxblitfunc.cpp
index 6efce719a..d5e3268c7 100644
--- a/src/resources/sdlgfxblitfunc.cpp
+++ b/src/resources/sdlgfxblitfunc.cpp
@@ -430,7 +430,7 @@ static int _SDL_gfxBlitRGBACall(const SDL_Surface *const src,
return 1;
}
- return (0);
+ return 0;
}
int SDLgfxBlitRGBA(SDL_Surface *const src,
diff --git a/src/resources/sprite/animatedsprite.cpp b/src/resources/sprite/animatedsprite.cpp
index 4905f2f3e..30eb4dda3 100644
--- a/src/resources/sprite/animatedsprite.cpp
+++ b/src/resources/sprite/animatedsprite.cpp
@@ -237,8 +237,8 @@ bool AnimatedSprite::updateCurrentAnimation(const unsigned int time) restrict2
mFrameIndex = 0;
mFrame = &mAnimation->mFrames[mFrameIndex];
- if ((mFrame->type == FrameType::LABEL &&
- !mFrame->nextAction.empty()))
+ if (mFrame->type == FrameType::LABEL &&
+ !mFrame->nextAction.empty())
{
fail = false;
}
diff --git a/src/resources/wallpaper.cpp b/src/resources/wallpaper.cpp
index dfaca1f30..9e81f53a7 100644
--- a/src/resources/wallpaper.cpp
+++ b/src/resources/wallpaper.cpp
@@ -80,7 +80,8 @@ static bool wallpaperCompare(const WallpaperData &a, const WallpaperData &b)
const int aa = a.width * a.height;
const int ab = b.width * b.height;
- return (aa > ab || (aa == ab && a.width > b.width));
+ return aa > ab ||
+ (aa == ab && a.width > b.width);
}
void Wallpaper::loadWallpapers()
diff --git a/src/utils/stringutils.cpp b/src/utils/stringutils.cpp
index ecbd7de19..4f3a156cc 100644
--- a/src/utils/stringutils.cpp
+++ b/src/utils/stringutils.cpp
@@ -157,7 +157,10 @@ int compareStrI(const std::string &a, const std::string &b)
bool isWordSeparator(const signed char chr)
{
- return (chr == ' ' || chr == ',' || chr == '.' || chr == '"');
+ return chr == ' ' ||
+ chr == ',' ||
+ chr == '.' ||
+ chr == '"';
}
const std::string findSameSubstring(const std::string &restrict str1,