feat: modify visibility of private functions and variables

Author: 0xAustrian

src/contracts/L2Resolver.sol

@@ -27,10 +27,10 @@ contract L2Resolver is IL2Resolver, IERC165 {
   bytes32 public constant PARENT_NODE_NAMEHASH = 0xe0266f32655869693dc63423876e5a7367f6e55552b718bdf9c96d0b31f8155d;
 
   /// @notice The hash of the ".l2.eth" suffix.
-  bytes32 private constant _L2_ETH_SUFFIX_HASH = keccak256(bytes('.l2.eth'));
+  bytes32 internal constant _L2_ETH_SUFFIX_HASH = keccak256(bytes('.l2.eth'));
 
   /// @notice The length of the ".l2.eth" suffix.
-  uint256 private constant _L2_ETH_SUFFIX_LENGTH = 7;
+  uint256 internal constant _L2_ETH_SUFFIX_LENGTH = 7;
 
   /// @inheritdoc IL2Resolver
   IENS public immutable ENS_REGISTRY;
@@ -123,7 +123,7 @@ contract L2Resolver is IL2Resolver, IERC165 {
    * @param _key The key to store the data under.
    * @param _value The data to store under the key.
    */
-  function _setRecord(bytes32 _node, string calldata _key, bytes calldata _value) private {
+  function _setRecord(bytes32 _node, string calldata _key, bytes calldata _value) internal {
     if (_node == REVERSE_LOOKUP_NODE) {
       // If the node is the reverse lookup node, then we validate the caller is the owner or an approved operator
       // of _value which is the encoded domain of the 3LD.
@@ -143,7 +143,7 @@ contract L2Resolver is IL2Resolver, IERC165 {
    * @param _caller The address to check.
    * @param _node The node to check.
    */
-  function _authenticateCaller(address _caller, bytes32 _node) private view {
+  function _authenticateCaller(address _caller, bytes32 _node) internal view {
     address _owner = _getOwner(_node);
     if (_owner != address(0)) {
       // If the node has an owner, then the caller must be an approved operator or the owner
@@ -164,7 +164,7 @@ contract L2Resolver is IL2Resolver, IERC165 {
    * @param _node The node to get the owner of.
    * @return _owner The owner of the node. Returns address(0) if the node is not owned by the ENSRegistry.
    */
-  function _getOwner(bytes32 _node) private view returns (address _owner) {
+  function _getOwner(bytes32 _node) internal view returns (address _owner) {
     _owner = ENS_REGISTRY.owner(_node);
     if (_owner == address(NAME_WRAPPER)) {
       _owner = NAME_WRAPPER.ownerOf(uint256(_node));
@@ -177,7 +177,7 @@ contract L2Resolver is IL2Resolver, IERC165 {
    * @param _owner The owner of the node.
    * @return _approved True if the _caller is an owner or an approved operator for the _owner, false otherwise.
    */
-  function _isApprovedOperatorOrOwner(address _caller, address _owner) private view returns (bool _approved) {
+  function _isApprovedOperatorOrOwner(address _caller, address _owner) internal view returns (bool _approved) {
     if (_owner == _caller) return true;
     _approved = ENS_REGISTRY.isApprovedForAll(_owner, _caller) || NAME_WRAPPER.isApprovedForAll(_owner, _caller);
   }