KyberNetwork / smart-contracts

Main smart contracts for Kyber Network, including the main platform contract, reserve contracts etc.
https://kyber.network
MIT License
378 stars 340 forks source link

storage validate if reserve is listed for token #968

Closed ducquangkstn closed 4 years ago

ducquangkstn commented 4 years ago

What did this PR do:

Gas report

┌─────────────────────────────────────────────────────────┬──────────┬─────────┬───────┐
│                         (index)                         │ Katalyst │ current │ diff  │
├─────────────────────────────────────────────────────────┼──────────┼─────────┼───────┤
│      t2e trade, type: SPLIT, fee: 123, reserves: 3      │  576101  │ 579550  │ 3449  │
│      e2t trade, type: SPLIT, fee: 123, reserves: 3      │  561132  │ 564591  │ 3459  │
│      t2t trade, type: SPLIT, fee: 123, reserves: 3      │  979449  │ 986706  │ 7257  │
│     t2e trade, type: NO HINT, fee: 123, reserves: 3     │  501635  │ 507031  │ 5396  │
│     e2t trade, type: NO HINT, fee: 123, reserves: 3     │  540672  │ 546100  │ 5428  │
│     t2t trade, type: NO HINT, fee: 123, reserves: 3     │  918403  │ 929274  │ 10871 │
│     t2e trade, type: MASK_IN, fee: 123, reserves: 3     │  393024  │ 396461  │ 3437  │
│     e2t trade, type: MASK_IN, fee: 123, reserves: 3     │  416721  │ 420168  │ 3447  │
│     t2t trade, type: MASK_IN, fee: 123, reserves: 3     │  691987  │ 698884  │ 6897  │
│    t2e trade, type: MASK_OUT, fee: 123, reserves: 2     │  404575  │ 408024  │ 3449  │
│    e2t trade, type: MASK_OUT, fee: 123, reserves: 2     │  427702  │ 431182  │ 3480  │
│    t2t trade, type: MASK_OUT, fee: 123, reserves: 2     │  715216  │ 722146  │ 6930  │
│  t2e trade maxdest, type: SPLIT, fee: 123, reserves: 3  │  560809  │ 564606  │ 3797  │
│  e2t trade maxdest, type: SPLIT, fee: 123, reserves: 3  │  574967  │ 578774  │ 3807  │
│  t2t trade maxdest, type: SPLIT, fee: 123, reserves: 3  │  998729  │ 1005985 │ 7256  │
│ t2e trade maxdest, type: NO HINT, fee: 123, reserves: 3 │  513066  │ 518520  │ 5454  │
│ e2t trade maxdest, type: NO HINT, fee: 123, reserves: 3 │  551079  │ 556507  │ 5428  │
│ t2t trade maxdest, type: NO HINT, fee: 123, reserves: 3 │  950683  │ 961321  │ 10638 │
└─────────────────────────────────────────────────────────┴──────────┴─────────┴───────┘