Remove double negation for readability

This commit is contained in:
Sergio Martins
2019-07-05 11:11:56 +01:00
parent e9128da69f
commit a7e3b5a3a2
2 changed files with 15 additions and 1 deletions

View File

@@ -269,6 +269,19 @@ bool Anchor::hasItems(Anchor::Side side) const
}
}
bool Anchor::onlyHasPlaceholderItems(Anchor::Side side) const
{
auto &items = side == Side1 ? m_side1Items
: m_side2Items;
for (Item *item : items) {
if (!item->isPlaceholder())
return false;
}
return true;
}
bool Anchor::hasNonPlaceholderItems(Anchor::Side side) const
{
auto &items = side == Side1 ? m_side1Items

View File

@@ -186,7 +186,8 @@ public:
bool isEmpty() const { return !hasItems(Side1) && !hasItems(Side2); }
bool hasItems(Side) const;
bool hasNonPlaceholderItems(Side) const;
bool shouldFollow() const{ return !isStatic() && (!hasNonPlaceholderItems(Side1) || !hasNonPlaceholderItems(Side2)) && !isFollowing(); }
bool onlyHasPlaceholderItems(Anchor::Side side) const;
bool shouldFollow() const { return !isStatic() && (onlyHasPlaceholderItems(Side1) || onlyHasPlaceholderItems(Side2)) && !isFollowing(); }
bool containsItem(const Item *w, Side side) const;
bool isStaticOrFollowsStatic() const;