fix: add braces to one-line if statements in ERC721EnumerableFacet

panditdhamdhere · panditdhamdhere · commit 31871480c25d · 2025-12-05T12:50:19.000+05:30
Fix style guide violation by adding braces around one-line if statements
that call revert. This ensures compliance with Compose's coding standards
which require all if statements to use braces.

Fixes two instances in safeTransferFrom functions (lines 331 and 354).
diff --git a/src/token/ERC721/ERC721Enumerable/ERC721EnumerableFacet.sol b/src/token/ERC721/ERC721Enumerable/ERC721EnumerableFacet.sol
@@ -328,7 +328,9 @@ contract ERC721EnumerableFacet {
                     revert ERC721InvalidReceiver(_to);
                 }
             } catch (bytes memory reason) {
-                if (reason.length == 0) revert ERC721InvalidReceiver(_to);
+                if (reason.length == 0) {
+                    revert ERC721InvalidReceiver(_to);
+                }
                 assembly ("memory-safe") {
                     revert(add(reason, 0x20), mload(reason))
                 }
@@ -351,7 +353,9 @@ contract ERC721EnumerableFacet {
                     revert ERC721InvalidReceiver(_to);
                 }
             } catch (bytes memory reason) {
-                if (reason.length == 0) revert ERC721InvalidReceiver(_to);
+                if (reason.length == 0) {
+                    revert ERC721InvalidReceiver(_to);
+                }
                 assembly ("memory-safe") {
                     revert(add(reason, 0x20), mload(reason))
                 }

Original file line number	Diff line number	Diff line change
`@@ -328,7 +328,9 @@ contract ERC721EnumerableFacet {`
`328`	`328`	`revert ERC721InvalidReceiver(_to);`
`329`	`329`	`}`
`330`	`330`	`} catch (bytes memory reason) {`
`331`		`- if (reason.length == 0) revert ERC721InvalidReceiver(_to);`
	`331`	`+ if (reason.length == 0) {`
	`332`	`+ revert ERC721InvalidReceiver(_to);`
	`333`	`+ }`
`332`	`334`	`assembly ("memory-safe") {`
`333`	`335`	`revert(add(reason, 0x20), mload(reason))`
`334`	`336`	`}`
`@@ -351,7 +353,9 @@ contract ERC721EnumerableFacet {`
`351`	`353`	`revert ERC721InvalidReceiver(_to);`
`352`	`354`	`}`
`353`	`355`	`} catch (bytes memory reason) {`
`354`		`- if (reason.length == 0) revert ERC721InvalidReceiver(_to);`
	`356`	`+ if (reason.length == 0) {`
	`357`	`+ revert ERC721InvalidReceiver(_to);`
	`358`	`+ }`
`355`	`359`	`assembly ("memory-safe") {`
`356`	`360`	`revert(add(reason, 0x20), mload(reason))`
`357`	`361`	`}`