From 31871480c25dfd6590fa7ba73c87ade4980ad0ed Mon Sep 17 00:00:00 2001 From: panditdhamdhere Date: Fri, 5 Dec 2025 12:50:19 +0530 Subject: [PATCH] fix: add braces to one-line if statements in ERC721EnumerableFacet 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). --- .../ERC721/ERC721Enumerable/ERC721EnumerableFacet.sol | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/token/ERC721/ERC721Enumerable/ERC721EnumerableFacet.sol b/src/token/ERC721/ERC721Enumerable/ERC721EnumerableFacet.sol index 0d0671c1..c0bf1e4f 100644 --- 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)) }