search

primitivefinance / DFMM

Smart contracts of the DFMM protocol
https://www.primitive.xyz/blog/dfmm_protocol
18 stars 6 forks source link

feat: Solvers clean #116

Closed 0xJepsen closed 3 months ago

0xJepsen commented 3 months ago

Closes #115

I'm sure this can be cleaner but this does what I want.

Alexangelj commented 3 months ago

I also feel like this should be a PR outside of the audit fixes. I imagine we will continue to edit the solvers as we need, so these will continue to be "unaudited", but i'll leave this up to @clemlak to decide on what to do

0xJepsen commented 3 months ago

Sure yeah review all you need.

clemlak commented 3 months ago

I agree with Alex, maybe we should merge this into another branch later because the fix/spearbit-audit branch might be used by the auditors to check the fixes.

0xJepsen commented 3 months ago

Closed in favor of #130