Skip to content

Commit

Permalink
chore: simplify comments
Browse files Browse the repository at this point in the history
  • Loading branch information
PaulRBerg committed Nov 26, 2023
1 parent 50ceaab commit d3d1e4e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
4 changes: 1 addition & 3 deletions src/SablierV2LockupDynamic.sol
Original file line number Diff line number Diff line change
Expand Up @@ -515,9 +515,7 @@ contract SablierV2LockupDynamic is
// Log the cancellation.
emit ISablierV2Lockup.CancelLockupStream(streamId, sender, recipient, asset, senderAmount, recipientAmount);

// Emits an ERC-4906 event to trigger an update of the NFT metadata. The `updateMetadata` modifier isn't used
// here to avoid DOS attacks by malicious recipient contracts, which could force excessive gas costs on the
// sender.
// Emits an ERC-4906 event to trigger an update of the NFT metadata.
emit MetadataUpdate({ _tokenId: streamId });

// Interactions: if the recipient is a contract, try to invoke the cancel hook on the recipient without
Expand Down
4 changes: 1 addition & 3 deletions src/SablierV2LockupLinear.sol
Original file line number Diff line number Diff line change
Expand Up @@ -430,9 +430,7 @@ contract SablierV2LockupLinear is
// Log the cancellation.
emit ISablierV2Lockup.CancelLockupStream(streamId, sender, recipient, asset, senderAmount, recipientAmount);

// Emits an ERC-4906 event to trigger an update of the NFT metadata. The `updateMetadata` modifier isn't used
// here to avoid DOS attacks by malicious recipient contracts, which could force excessive gas costs on the
// sender.
// Emits an ERC-4906 event to trigger an update of the NFT metadata.
emit MetadataUpdate({ _tokenId: streamId });

// Interactions: if the recipient is a contract, try to invoke the cancel hook on the recipient without
Expand Down

0 comments on commit d3d1e4e

Please sign in to comment.