Skip to content
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

There are so many bugs in the code, is that on purpose so that the developers get a chance to really understand it by fixing the code ? #359

Open
kuchaguangjie opened this issue Mar 9, 2025 · 3 comments

Comments

@kuchaguangjie
Copy link

For example, this line:

it should be checked_div(), not checked_mul, right?

@kuchaguangjie kuchaguangjie changed the title There are so many bugs in the code, is that on purpose so that make the developer really understand the code by fixing it ? There are so many bugs in the code, is that on purpose so that the developers get a chance to really understand it by fixing the code ? Mar 9, 2025
@jacobcreech
Copy link
Contributor

@kuchaguangjie it's not meant to have bugs in the code. Any other programs you find problematic? We'll go through them

@kuchaguangjie
Copy link
Author

quite some bugs I've found in various sub projects, and to make it work with newest anchor version, almost need always do refactor.

@kuchaguangjie
Copy link
Author

Another possible error in the same repo is at:

According to the code, the formula for output is :

  • taxed * b / (a + taxed) when swap_a == true
  • taxed * a / (b + taxed) when swap_a == false

That looks doesn't make sense, or does it ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants