Skip to content

Commit a010ae1

Browse files
fix(redundancy): removed redundant amountBytes var & isZeroAmount check (#31)
Co-authored-by: Mannan Goyal <mannangoyal@luganodes.com>
1 parent dea54ef commit a010ae1

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

src/Pectra.sol

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -159,14 +159,12 @@ contract Pectra {
159159
continue;
160160
}
161161

162-
if (!isZeroAmount && data[i].amount > MAX_WITHDRAWAL_AMOUNT) {
162+
if (data[i].amount > MAX_WITHDRAWAL_AMOUNT) {
163163
emit ExecutionLayerExitFailed(FailureReason.AMOUNT_EXCEEDS_MAXIMUM, data[i].pubkey, data[i].amount);
164164
continue;
165165
}
166166

167-
bytes memory amountBytes = abi.encodePacked(data[i].amount);
168-
169-
bytes memory concatenated = abi.encodePacked(data[i].pubkey, amountBytes);
167+
bytes memory concatenated = abi.encodePacked(data[i].pubkey, data[i].amount);
170168
(bool success,) = exitTarget.call{value: exitFee}(concatenated);
171169
if (!success) {
172170
emit ExecutionLayerExitFailed(FailureReason.OPERATION_FAILED, data[i].pubkey, data[i].amount);

0 commit comments

Comments
 (0)