Comments (3)
This is a good suggestion, though I believe the rounding errors will be small enough that it won't really be an issue. Happy to be convinced otherwise!
from niftyswap.
Rounding errors should be small enough for most of the tokens, but isn’t unheard of ERC20 tokens that have no decimals and end up with a minimal fragmentation unit of 1 dollar or so, and if a token like that were to be traded on Niftyswap it would be inefficient.
Also, the rounding on getBuyPrice
could lead to lots of trades of round numbers to increase by 1 wei, maybe that could cause some confusion?
Last but not least, I think that is a simple fix that barely increases the gas usage of the contract.
from niftyswap.
ccc6e9d introduces a slightly difference method than commit above ^
from niftyswap.
Related Issues (20)
- Front Running Exchange Creation
- C1 - Pool can be drained if `base` matches `token`
- L2 - An error on `base` or `token` could leave the whole pool frozen HOT 2
- N1 - Token trades are not easily composable HOT 2
- N3 - Assembly code for `functionSignature` retrieval can be removed
- N4 - self-deposit isn't enforced HOT 1
- L3 - Minimum base provided can be circumvent HOT 5
- L4 - `addLiquidity` can round in favor of new liquidity provider HOT 2
- N5 - Definition, and assignment of arrays can be optimized
- N6 - Functions mutability can be restricted to pure
- N7 - `refundBase` isn’t sent to the seller HOT 1
- N8 - Minted liquidity tokens can round to zero HOT 4
- Bug Bounty: Pre-audit HOT 1
- Support for ERC20::ERC1155 Niftyswap pool HOT 7
- Adding creatorFee to Niftyswap HOT 7
- Updating solidity compiler version to ^0.8.0 HOT 3
- TypeError: Member "buyTokens" not found or not visible after argument-dependent lookup in address HOT 4
- Export method signatures and data types for npm package consumers
- Orderbook: Collectible Detail functionality integrations
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from niftyswap.