docs: update EIP712 upgradeable comment in patch file

Fix misleading comment in EIP712Upgradeable by updating the comment in the transpilation patch file. The previous comment incorrectly referred to immutable values that don't exist in the upgradeable version.

The patch file has also been cleaned of trailing whitespace.

Author: naevern

.changeset/smart-teams-raise.md

@@ -0,0 +1,2 @@
+---
+---

scripts/upgradeable/upgradeable.patch

@@ -1,6 +1,6 @@
 diff --git a/.github/ISSUE_TEMPLATE/bug_report.md b/.github/ISSUE_TEMPLATE/bug_report.md
 deleted file mode 100644
-index 35ad097ff..000000000
+index 35ad097f..00000000
 --- a/.github/ISSUE_TEMPLATE/bug_report.md
 +++ /dev/null
 @@ -1,21 +0,0 @@
@@ -26,7 +26,7 @@ index 35ad097ff..000000000
 -
 -<!-- We will be able to better help if you provide a minimal example that triggers the bug. -->
 diff --git a/.github/ISSUE_TEMPLATE/config.yml b/.github/ISSUE_TEMPLATE/config.yml
-index 4018cef29..d343a53d8 100644
+index 4018cef2..d343a53d 100644
 --- a/.github/ISSUE_TEMPLATE/config.yml
 +++ b/.github/ISSUE_TEMPLATE/config.yml
 @@ -1,4 +1,8 @@
@@ -40,7 +40,7 @@ index 4018cef29..d343a53d8 100644
      about: Ask in the OpenZeppelin Forum
 diff --git a/.github/ISSUE_TEMPLATE/feature_request.md b/.github/ISSUE_TEMPLATE/feature_request.md
 deleted file mode 100644
-index ff596b0c3..000000000
+index ff596b0c..00000000
 --- a/.github/ISSUE_TEMPLATE/feature_request.md
 +++ /dev/null
 @@ -1,14 +0,0 @@
@@ -59,48 +59,48 @@ index ff596b0c3..000000000
 -<!-- Make sure that you have reviewed the OpenZeppelin Contracts Contributor Guidelines. -->
 -<!-- https://github.com/OpenZeppelin/openzeppelin-contracts/blob/master/CONTRIBUTING.md -->
 diff --git a/README.md b/README.md
-index 60d0a430a..0e4f91a6d 100644
+index 60d0a430..0e4f91a6 100644
 --- a/README.md
 +++ b/README.md
 @@ -19,6 +19,9 @@
  > [!IMPORTANT]
  > OpenZeppelin Contracts uses semantic versioning to communicate backwards compatibility of its API and storage layout. For upgradeable contracts, the storage layout of different major versions should be assumed incompatible, for example, it is unsafe to upgrade from 4.9.3 to 5.0.0. Learn more at [Backwards Compatibility](https://docs.openzeppelin.com/contracts/backwards-compatibility).
- 
+
 ++> [!NOTE]
 ++> You are looking at the upgradeable variant of OpenZeppelin Contracts. Be sure to review the documentation on [Using OpenZeppelin Contracts with Upgrades](https://docs.openzeppelin.com/contracts/upgradeable).
 ++
  ## Overview
- 
+
  ### Installation
 @@ -26,7 +29,7 @@
  #### Hardhat (npm)
- 
+
  ```
 -$ npm install @openzeppelin/contracts
 +$ npm install @openzeppelin/contracts-upgradeable
  ```
- 
+
  #### Foundry (git)
 @@ -38,10 +41,10 @@ $ npm install @openzeppelin/contracts
  > Foundry installs the latest version initially, but subsequent `forge update` commands will use the `master` branch.
- 
+
  ```
 -$ forge install OpenZeppelin/openzeppelin-contracts
 +$ forge install OpenZeppelin/openzeppelin-contracts-upgradeable
  ```
- 
+
 -Add `@openzeppelin/contracts/=lib/openzeppelin-contracts/contracts/` in `remappings.txt.`
 +Add `@openzeppelin/contracts-upgradeable/=lib/openzeppelin-contracts-upgradeable/contracts/` in `remappings.txt.`
- 
+
  ### Usage
- 
+
 @@ -50,10 +53,11 @@ Once installed, you can use the contracts in the library by importing them:
  ```solidity
  pragma solidity ^0.8.20;
- 
+
 -import {ERC721} from "@openzeppelin/contracts/token/ERC721/ERC721.sol";
 +import {ERC721Upgradeable} from "@openzeppelin/contracts-upgradeable/token/ERC721/ERC721Upgradeable.sol";
- 
+
 -contract MyCollectible is ERC721 {
 -    constructor() ERC721("MyCollectible", "MCO") {
 +contract MyCollectible is ERC721Upgradeable {
@@ -110,15 +110,15 @@ index 60d0a430a..0e4f91a6d 100644
  }
  ```
 diff --git a/contracts/package.json b/contracts/package.json
-index 3682eadeb..4f870d094 100644
+index 70ae73bc..ef659873 100644
 --- a/contracts/package.json
 +++ b/contracts/package.json
 @@ -1,5 +1,5 @@
  {
 -  "name": "@openzeppelin/contracts",
 +  "name": "@openzeppelin/contracts-upgradeable",
    "description": "Secure Smart Contract library for Solidity",
-   "version": "5.2.0",
+   "version": "5.3.0",
    "files": [
 @@ -13,7 +13,7 @@
    },
@@ -140,30 +140,35 @@ index 3682eadeb..4f870d094 100644
 +  }
  }
 diff --git a/contracts/utils/cryptography/EIP712.sol b/contracts/utils/cryptography/EIP712.sol
-index bcb67c87a..7195c3bbd 100644
+index c39954e3..fe681f87 100644
 --- a/contracts/utils/cryptography/EIP712.sol
 +++ b/contracts/utils/cryptography/EIP712.sol
 @@ -4,7 +4,6 @@
  pragma solidity ^0.8.20;
- 
+
  import {MessageHashUtils} from "./MessageHashUtils.sol";
 -import {ShortStrings, ShortString} from "../ShortStrings.sol";
  import {IERC5267} from "../../interfaces/IERC5267.sol";
- 
+
  /**
-@@ -28,30 +27,18 @@ import {IERC5267} from "../../interfaces/IERC5267.sol";
-  * NOTE: In the upgradeable version of this contract, the cached values will correspond to the address, and the domain
-  * separator of the implementation contract. This will cause the {_domainSeparatorV4} function to always rebuild the
-  * separator from the immutable values, which is cheaper than accessing a cached version in cold storage.
+@@ -25,33 +24,20 @@ import {IERC5267} from "../../interfaces/IERC5267.sol";
+  * NOTE: This contract implements the version of the encoding known as "v4", as implemented by the JSON RPC method
+  * https://docs.metamask.io/guide/signing-data.html[`eth_signTypedDataV4` in MetaMask].
+  *
+- * NOTE: In the upgradeable version of this contract, the cached values will correspond to the address, and the domain
+- * separator of the implementation contract. This will cause the {_domainSeparatorV4} function to always rebuild the
+- * separator from the immutable values, which is cheaper than accessing a cached version in cold storage.
 - *
 - * @custom:oz-upgrades-unsafe-allow state-variable-immutable
++ * NOTE: The upgradeable version of this contract does not use an immutable cache and recomputes the domain separator
++ * each time {_domainSeparatorV4} is called. That is cheaper than accessing a cached version in cold storage.
   */
  abstract contract EIP712 is IERC5267 {
 -    using ShortStrings for *;
 -
      bytes32 private constant TYPE_HASH =
          keccak256("EIP712Domain(string name,string version,uint256 chainId,address verifyingContract)");
- 
+
 -    // Cache the domain separator as an immutable value, but also store the chain id that it corresponds to, in order to
 -    // invalidate the cached domain separator if the chain id changes.
 -    bytes32 private immutable _cachedDomainSeparator;
@@ -174,7 +179,7 @@ index bcb67c87a..7195c3bbd 100644
      bytes32 private immutable _hashedName;
 +    /// @custom:oz-renamed-from _HASHED_VERSION
      bytes32 private immutable _hashedVersion;
- 
+
 -    ShortString private immutable _name;
 -    ShortString private immutable _version;
 -    // slither-disable-next-line constable-states
@@ -183,10 +188,10 @@ index bcb67c87a..7195c3bbd 100644
 -    string private _versionFallback;
 +    string private _name;
 +    string private _version;
- 
+
      /**
       * @dev Initializes the domain separator and parameter caches.
-@@ -66,29 +53,23 @@ abstract contract EIP712 is IERC5267 {
+@@ -66,29 +52,23 @@ abstract contract EIP712 is IERC5267 {
       * contract upgrade].
       */
      constructor(string memory name, string memory version) {
@@ -205,7 +210,7 @@ index bcb67c87a..7195c3bbd 100644
 +        _hashedName = 0;
 +        _hashedVersion = 0;
      }
- 
+
      /**
       * @dev Returns the domain separator for the current chain.
       */
@@ -217,14 +222,14 @@ index bcb67c87a..7195c3bbd 100644
 -        }
 +        return _buildDomainSeparator();
      }
- 
+
      function _buildDomainSeparator() private view returns (bytes32) {
 -        return keccak256(abi.encode(TYPE_HASH, _hashedName, _hashedVersion, block.chainid, address(this)));
 +        return keccak256(abi.encode(TYPE_HASH, _EIP712NameHash(), _EIP712VersionHash(), block.chainid, address(this)));
      }
- 
+
      /**
-@@ -127,6 +108,10 @@ abstract contract EIP712 is IERC5267 {
+@@ -125,6 +105,10 @@ abstract contract EIP712 is IERC5267 {
              uint256[] memory extensions
          )
      {
@@ -235,7 +240,7 @@ index bcb67c87a..7195c3bbd 100644
          return (
              hex"0f", // 01111
              _EIP712Name(),
-@@ -141,22 +126,62 @@ abstract contract EIP712 is IERC5267 {
+@@ -139,22 +123,62 @@ abstract contract EIP712 is IERC5267 {
      /**
       * @dev The name parameter for the EIP712 domain.
       *
@@ -250,7 +255,7 @@ index bcb67c87a..7195c3bbd 100644
 +    function _EIP712Name() internal view virtual returns (string memory) {
 +        return _name;
      }
- 
+
      /**
       * @dev The version parameter for the EIP712 domain.
       *
@@ -309,7 +314,7 @@ index bcb67c87a..7195c3bbd 100644
      }
  }
 diff --git a/package.json b/package.json
-index f9e7d9205..c35020d51 100644
+index eeeaf0bc..65581c54 100644
 --- a/package.json
 +++ b/package.json
 @@ -34,7 +34,7 @@
@@ -322,15 +327,15 @@ index f9e7d9205..c35020d51 100644
    "keywords": [
      "solidity",
 diff --git a/remappings.txt b/remappings.txt
-index 304d1386a..a1cd63bee 100644
+index 304d1386..a1cd63be 100644
 --- a/remappings.txt
 +++ b/remappings.txt
 @@ -1 +1,2 @@
 -@openzeppelin/contracts/=contracts/
 +@openzeppelin/contracts-upgradeable/=contracts/
 +@openzeppelin/contracts/=lib/openzeppelin-contracts/contracts/
 diff --git a/test/utils/cryptography/EIP712.test.js b/test/utils/cryptography/EIP712.test.js
-index 2b6e7fa97..268e0d29d 100644
+index 2b6e7fa9..268e0d29 100644
 --- a/test/utils/cryptography/EIP712.test.js
 +++ b/test/utils/cryptography/EIP712.test.js
 @@ -47,27 +47,6 @@ describe('EIP712', function () {
@@ -359,5 +364,5 @@ index 2b6e7fa97..268e0d29d 100644
 -          });
 -        }
        });
- 
+
        it('hash digest', async function () {