Remove more dead code

[harisang]

Follow-up to PR #592, and further addressing issue #560

[fhenneke]

The changes look good.

I have not tested this change. Could you add a description of how to test this change?

[fhenneke]

I assume it was intentional to change from the syntax used in other places to this approach?

        match network:
            case (
                Network.MAINNET
                | Network.GNOSIS
                | Network.ARBITRUM_ONE
                | Network.BASE
                | Network.AVALANCHE
                | Network.POLYGON
                | Network.BNB
            ):
                protocol_fee_safe = Address("0x22af3D38E50ddedeb7C47f36faB321eC3Bb72A76")
            case Network.LENS:
                protocol_fee_safe = Address("0x07e5292b5aac443B2C9473Ab51B53ce8BDC3317B")
            case _:
                raise ValueError(
                    f"No buffer accounting config set up for network {network}."
                )

That other approach does have the advantage of being very explicit and notifying of missing configuration.

If you want to go for what you did here, I think adding an additional check for network in Network would be welcome.

[harisang]

Good point. Pushed a fix

[harisang]

@fhenneke pushed some a small change, so pinging in case you want to have another look. Planning to merge later today

[fhenneke]

Looks good. (I have not tested the change and are not planning on it due to missing instuctions.)