Missing disableInitializers() call in the constructor

Description

When using Initializable.sol, it's a good practice calling disableInitializers() in the constructor, such that the implementation itself can't be initialized.

The call is missing in the Clearinghouse.sol, Endpoint.sol, OffchainExchange.sol, PerpEngine.sol, SpotEngine.sol, ClearinghouseLiq.sol and Verifier.sol (commented out).

ClearingHouse may be selfdestructed via the delegateCall().

Recommendation

Use:

constructor() {
    _disableInitializers();
}

Status

Acknowledged