♻️ Fix comments (#480)

src/tokens/ERC1155.sol

@@ -5,7 +5,8 @@ pragma solidity ^0.8.4;
 /// @author Solady (https://github.com/vectorized/solady/blob/main/src/tokens/ERC1155.sol)
 /// @author Modified from Solmate (https://github.com/transmissions11/solmate/blob/main/src/tokens/ERC1155.sol)
 /// @author Modified from OpenZeppelin (https://github.com/OpenZeppelin/openzeppelin-contracts/tree/master/contracts/token/ERC1155/ERC1155.sol)
-/// Note:
+///
+/// @dev Note:
 /// The ERC1155 standard allows for self-approvals.
 /// For performance, this implementation WILL NOT revert for such actions.
 /// Please add any checks with overrides if desired.

src/tokens/ERC20.sol

@@ -5,7 +5,8 @@ pragma solidity ^0.8.4;
 /// @author Solady (https://github.com/vectorized/solady/blob/main/src/tokens/ERC20.sol)
 /// @author Modified from Solmate (https://github.com/transmissions11/solmate/blob/main/src/tokens/ERC20.sol)
 /// @author Modified from OpenZeppelin (https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/token/ERC20/ERC20.sol)
-/// Note:
+///
+/// @dev Note:
 /// The ERC20 standard allows minting and transferring to and from the zero address,
 /// minting and transferring zero tokens, as well as self-approvals.
 /// For performance, this implementation WILL NOT revert for such actions.
@@ -251,7 +252,7 @@ abstract contract ERC20 {
 
     /// @dev Transfers `amount` tokens from `from` to `to`.
     ///
-    /// Note: does not update the allowance if it is the maximum uint256 value.
+    /// Note: Does not update the allowance if it is the maximum uint256 value.
     ///
     /// Requirements:
     /// - `from` must at least have `amount`.

src/tokens/ERC721.sol

@@ -5,7 +5,8 @@ pragma solidity ^0.8.4;
 /// @author Solady (https://github.com/vectorized/solady/blob/main/src/tokens/ERC721.sol)
 /// @author Modified from Solmate (https://github.com/transmissions11/solmate/blob/main/src/tokens/ERC721.sol)
 /// @author Modified from OpenZeppelin (https://github.com/OpenZeppelin/openzeppelin-contracts/tree/master/contracts/token/ERC721/ERC721.sol)
-/// Note:
+///
+/// @dev Note:
 /// The ERC721 standard allows for self-approvals.
 /// For performance, this implementation WILL NOT revert for such actions.
 /// Please add any checks with overrides if desired.
@@ -399,7 +400,7 @@ abstract contract ERC721 {
 
     /// @dev Returns the extra data for token `id`.
     /// Minting, transferring, burning a token will not change the extra data.
-    /// The extra data can be set on a non existent token.
+    /// The extra data can be set on a non-existent token.
     function _getExtraData(uint256 id) internal view virtual returns (uint96 result) {
         /// @solidity memory-safe-assembly
         assembly {
@@ -411,7 +412,7 @@ abstract contract ERC721 {
 
     /// @dev Sets the extra data for token `id` to `value`.
     /// Minting, transferring, burning a token will not change the extra data.
-    /// The extra data can be set on a non existent token.
+    /// The extra data can be set on a non-existent token.
     function _setExtraData(uint256 id, uint96 value) internal virtual {
         /// @solidity memory-safe-assembly
         assembly {

src/utils/LibZip.sol

@@ -5,7 +5,8 @@ pragma solidity ^0.8.4;
 /// @author Solady (https://github.com/vectorized/solady/blob/main/src/utils/LibZip.sol)
 /// @author Calldata compression by clabby (https://github.com/clabby/op-kompressor)
 /// @author FastLZ by ariya (https://github.com/ariya/FastLZ)
-/// Note:
+///
+/// @dev Note:
 /// The accompanying solady.js library includes implementations of
 /// FastLZ and calldata operations for convenience.
 library LibZip {

src/utils/SafeTransferLib.sol

@@ -4,7 +4,11 @@ pragma solidity ^0.8.4;
 /// @notice Safe ETH and ERC20 transfer library that gracefully handles missing return values.
 /// @author Solady (https://github.com/vectorized/solady/blob/main/src/utils/SafeTransferLib.sol)
 /// @author Modified from Solmate (https://github.com/transmissions11/solmate/blob/main/src/utils/SafeTransferLib.sol)
-/// @dev Caution! This library won't check that a token has code, responsibility is delegated to the caller.
+///
+/// @dev Note:
+/// - For ETH transfers, please use `forceSafeTransferETH` for gas griefing protection.
+/// - For ERC20s, this implementation won't check that a token has code,
+/// responsibility is delegated to the caller.
 library SafeTransferLib {
     /*´:°•.°+.*•´.*:˚.°*.˚•´.°:°•.°•.*•´.*:˚.°*.˚•´.°:°•.°+.*•´.*:*/
     /*                       CUSTOM ERRORS                        */
@@ -41,6 +45,9 @@ library SafeTransferLib {
 
     /// @dev Sends `amount` (in wei) ETH to `to`.
     /// Reverts upon failure.
+    ///
+    /// Note: This implementation does NOT protect against gas griefing.
+    /// Please use `forceSafeTransferETH` for gas griefing protection.
     function safeTransferETH(address to, uint256 amount) internal {
         /// @solidity memory-safe-assembly
         assembly {
@@ -211,7 +218,7 @@ library SafeTransferLib {
 
             // Store the function selector of `transferFrom(address,address,uint256)`.
             mstore(0x00, 0x23b872dd)
-            // The `amount` argument is already written to the memory word at 0x6c.
+            // The `amount` argument is already written to the memory word at 0x60.
             amount := mload(0x60)
 
             if iszero(

src/utils/SignatureCheckerLib.sol

@@ -6,7 +6,7 @@ pragma solidity ^0.8.4;
 /// @author Solady (https://github.com/vectorized/solady/blob/main/src/utils/SignatureCheckerLib.sol)
 /// @author Modified from OpenZeppelin (https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/contracts/utils/cryptography/SignatureChecker.sol)
 ///
-/// Note: unlike ECDSA signatures, contract signatures are revocable.
+/// @dev Note: Unlike ECDSA signatures, contract signatures are revocable.
 library SignatureCheckerLib {
     /*´:°•.°+.*•´.*:˚.°*.˚•´.°:°•.°•.*•´.*:˚.°*.˚•´.°:°•.°+.*•´.*:*/
     /*                         CONSTANTS                          */