[L-04] Mismatch between NatSpec and the actual code

[madMax92221]

The RiskPoolERC20 contract defines empty constructor with no parameters, which is not necessary and only hinders code readability. According to the Solidity docs on constructors: “If there is no constructor, the contract will assume the default constructor, which is equivalent to constructor() public {}“.

Although there are comments above the empty defined constructor:

/**
     * @dev Sets the values for {name} and {symbol}.
     *
     * The default value of {decimals} is 18. To select a different value for
     * {decimals} you should overload it.
     *
     * All two of these values are immutable: they can only be set once during
     * construction.
     */
    constructor() {}

Either define a constructor that the comments describe or delete the constructor declaration and the comments above it.

[wankhede04]

fixed at commit ab6376e