-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: position/liquidity_amounts #160
refactor: position/liquidity_amounts #160
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1 comment for variable name convension
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! The Get/compute convention here is a great way to demonstrate golang coding style and easier to read. Left a minor comment on X96
suffixes, could be handled in a future PR.
Description
Elimination of Duplicate Code: Implemented a logic to automatically swap the values of
sqrtRatioA
andsqrtRatioB
using theswapIfGreater
function without duplication.Function Name Change: Functions that are only used in the
liquidity_amounts.gno
file have been renamed tocomputeXXX
, while functions called from other places have been prefixed withGetXXX
."