F-2024-0016·dead-code
Redundant function getAssetsExtended
TL;DR
getAssetsExtended is a thin pass-through over getAssets and adds no functionality, so it can be removed without losing capability.
Severity
INFO
Impact
LOW
Likelihood
LOW
Method
MManual review
CAT.
Complexity
LOW
Exploitability
LOW
02Section · Description
Description
The getAssetsExtended function is redundant because it simply calls the getAssets function without adding any additional functionality. Both functions return the same result.
solidity
function getAssets() public view returns (uint256[] memory) {uint256[] memory totalAssetsResult = new uint256[](tokenAddresses.length);for (uint256 i = 0; i < tokenAddresses.length; i++) {totalAssetsResult[i] = totalAssets[tokenAddresses[i]];}return totalAssetsResult;}function getAssetsExtended() virtual public view returns (uint256[] memory) {return getAssets();}
03Section · Recommendation
Recommendation
Remove the getAssetsExtended function as it does not provide any additional value beyond what getAssets already offers.
