Skip to content

Commit

Permalink
Fabric: Fixed a bug in LayoutMetrics::operator==
Browse files Browse the repository at this point in the history
Summary: Trivial. We missed `pointScaleFactor`.

Reviewed By: mdvacca

Differential Revision: D10112051

fbshipit-source-id: 980b8c310fbb3701008765509dce5b6e61852c0e
  • Loading branch information
shergin authored and facebook-github-bot committed Sep 28, 2018
1 parent 9edf635 commit 5ce2c01
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions ReactCommon/fabric/core/layout/LayoutMetrics.h
Original file line number Diff line number Diff line change
Expand Up @@ -31,13 +31,13 @@ struct LayoutMetrics {
};
}

bool operator ==(const LayoutMetrics& rhs) const {
bool operator==(const LayoutMetrics &rhs) const {
return
std::tie(this->frame, this->contentInsets, this->borderWidth, this->displayType, this->layoutDirection) ==
std::tie(rhs.frame, rhs.contentInsets, rhs.borderWidth, rhs.displayType, rhs.layoutDirection);
std::tie(this->frame, this->contentInsets, this->borderWidth, this->displayType, this->layoutDirection, this->pointScaleFactor) ==
std::tie(rhs.frame, rhs.contentInsets, rhs.borderWidth, rhs.displayType, rhs.layoutDirection, this->pointScaleFactor);
}

bool operator !=(const LayoutMetrics& rhs) const {
bool operator!=(const LayoutMetrics &rhs) const {
return !(*this == rhs);
}
};
Expand Down

0 comments on commit 5ce2c01

Please sign in to comment.