Skip to content

Commit 3302858

Browse files
sammy-SCfacebook-github-bot
authored andcommitted
rename getters and mark them as const+noexcept in ValueAnimatedNode
Summary: changelog: [internal] rename methods to reflect that they are getters and mark them as noexcept const Reviewed By: rshest Differential Revision: D75217535
1 parent b34e562 commit 3302858

20 files changed

Lines changed: 56 additions & 52 deletions

packages/react-native/ReactCxxPlatform/react/renderer/animated/NativeAnimatedNodesManager.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -72,7 +72,7 @@ NativeAnimatedNodesManager::~NativeAnimatedNodesManager() noexcept {
7272
std::optional<double> NativeAnimatedNodesManager::getValue(Tag tag) noexcept {
7373
auto node = getAnimatedNode<ValueAnimatedNode>(tag);
7474
if (node) {
75-
return node->value();
75+
return node->getValue();
7676
} else {
7777
LOG(WARNING)
7878
<< "Cannot get value from AnimatedNode, it's not a ValueAnimatedNode";

packages/react-native/ReactCxxPlatform/react/renderer/animated/drivers/AnimationDriver.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ void AnimationDriver::stopAnimation(bool /*ignoreCompletedHandlers*/) {
5353
std::optional<double> value = std::nullopt;
5454
if (auto node =
5555
manager_->getAnimatedNode<ValueAnimatedNode>(animatedValueTag_)) {
56-
value = node->value();
56+
value = node->getValue();
5757
} else {
5858
LOG(ERROR)
5959
<< "animatedValueTag should be associated with a ValueAnimatedNode";

packages/react-native/ReactCxxPlatform/react/renderer/animated/drivers/DecayAnimationDriver.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ bool DecayAnimationDriver::update(double timeDeltaMs, bool restarting) {
4545
if (const auto node =
4646
manager_->getAnimatedNode<ValueAnimatedNode>(animatedValueTag_)) {
4747
if (restarting) {
48-
const auto value = node->rawValue();
48+
const auto value = node->getRawValue();
4949
if (!fromValue_.has_value()) {
5050
// First iteration, assign fromValue based on AnimatedValue
5151
fromValue_ = value;

packages/react-native/ReactCxxPlatform/react/renderer/animated/drivers/FrameAnimationDriver.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ bool FrameAnimationDriver::update(double timeDeltaMs, bool /*restarting*/) {
5858
if (auto node =
5959
manager_->getAnimatedNode<ValueAnimatedNode>(animatedValueTag_)) {
6060
if (!startValue_) {
61-
startValue_ = node->rawValue();
61+
startValue_ = node->getRawValue();
6262
}
6363

6464
const auto startIndex =

packages/react-native/ReactCxxPlatform/react/renderer/animated/drivers/SpringAnimationDriver.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ bool SpringAnimationDriver::update(double timeDeltaMs, bool restarting) {
8282
manager_->getAnimatedNode<ValueAnimatedNode>(animatedValueTag_)) {
8383
if (restarting) {
8484
if (!fromValue_.has_value()) {
85-
fromValue_ = node->rawValue();
85+
fromValue_ = node->getRawValue();
8686
} else {
8787
if (node->setRawValue(fromValue_.value())) {
8888
markNodeUpdated(node->tag());

packages/react-native/ReactCxxPlatform/react/renderer/animated/nodes/AdditionAnimatedNode.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ void AdditionAnimatedNode::update() {
2222
const auto node = manager_->getAnimatedNode<ValueAnimatedNode>(tag);
2323
react_native_assert(
2424
node && "Invalid node tag set as input for AdditionAnimatedNode");
25-
rawValue += node->value();
25+
rawValue += node->getValue();
2626
}
2727
setRawValue(rawValue);
2828
}

packages/react-native/ReactCxxPlatform/react/renderer/animated/nodes/ColorAnimatedNode.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,9 +23,10 @@ uint8_t getColorValue(
2323
bool isDecimal = false) {
2424
if (const auto node = manager.getAnimatedNode<ValueAnimatedNode>(nodeTag)) {
2525
if (isDecimal) {
26-
return std::clamp(static_cast<uint32_t>(node->value() * 255), 0u, 255u);
26+
return std::clamp(
27+
static_cast<uint32_t>(node->getValue() * 255), 0u, 255u);
2728
} else {
28-
return std::clamp(static_cast<uint32_t>(node->value()), 0u, 255u);
29+
return std::clamp(static_cast<uint32_t>(node->getValue()), 0u, 255u);
2930
}
3031
}
3132
return 0;

packages/react-native/ReactCxxPlatform/react/renderer/animated/nodes/DiffClampAnimatedNode.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,10 @@ DiffClampAnimatedNode::DiffClampAnimatedNode(
2727
void DiffClampAnimatedNode::update() {
2828
if (const auto node =
2929
manager_->getAnimatedNode<ValueAnimatedNode>(inputNodeTag_)) {
30-
const auto value = node->value();
30+
const auto value = node->getValue();
3131
const auto diff = value - lastValue_;
3232
lastValue_ = value;
33-
setRawValue(std::clamp(this->value() + diff, min_, max_));
33+
setRawValue(std::clamp(this->getValue() + diff, min_, max_));
3434
}
3535
}
3636

packages/react-native/ReactCxxPlatform/react/renderer/animated/nodes/DivisionAnimatedNode.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,9 +24,9 @@ void DivisionAnimatedNode::update() {
2424
react_native_assert(
2525
node && "Invalid node tag set as input for DivisionAnimatedNode");
2626
if (count == 0) {
27-
rawValue = node->value();
27+
rawValue = node->getValue();
2828
} else {
29-
rawValue /= node->value();
29+
rawValue /= node->getValue();
3030
}
3131

3232
count++;

packages/react-native/ReactCxxPlatform/react/renderer/animated/nodes/InterpolationAnimatedNode.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,9 @@ void InterpolationAnimatedNode::update() {
5454
if (const auto node =
5555
manager_->getAnimatedNode<ValueAnimatedNode>(parentTag_)) {
5656
if (isColorValue_) {
57-
setRawValue(interpolateColor(node->value()));
57+
setRawValue(interpolateColor(node->getValue()));
5858
} else {
59-
setRawValue(interpolateValue(node->value()));
59+
setRawValue(interpolateValue(node->getValue()));
6060
}
6161
}
6262
}

0 commit comments

Comments
 (0)