Originally I used unit8 to track owner index because I was packing this value. Later I switched to use SignatureWrapper and be more explicit, and rely on rollup compression to give me the 0s for free.
As this is now the case, we can simplify MultiOwnable to use uint256 for index, which allows us to remove addOwnerAtIndex at simplify the code.
Originally I used unit8 to track owner index because I was packing this value. Later I switched to use SignatureWrapper and be more explicit, and rely on rollup compression to give me the
0
s for free.As this is now the case, we can simplify MultiOwnable to use
uint256
for index, which allows us to removeaddOwnerAtIndex
at simplify the code.