cartesi / rollups-contracts

Smart Contracts for Cartesi Rollups
https://cartesi.github.io/rollups-contracts/
Apache License 2.0
17 stars 37 forks source link

Rename `LibOutputValidation` as `LibOutputValidityProof` #214

Closed guidanoli closed 5 months ago

guidanoli commented 6 months ago

📚 Context

Libraries in Solidity can specialize in working with specific data types. This paradigm is corroborated by the using <Library> for <DataType> syntax. In our code base, we have some of these library-type pairs:

One outlier is the LibOutputValidation library and OutputValidityProof struct. It may be due to the former getBitMaskPosition and validateInputIndexRange functions, which did not operate on OutputValidityProof values. But these functions have been removed, leaving only functions that do operate on such values.

✔️ Solution

Rename LibOutputValidation as LibOutputValidityProof.