Skip to content

Optimize inventory#2659

Merged
erikzhang merged 3 commits intoneo-project:developfrom
shargon:optimize-01
Apr 1, 2022
Merged

Optimize inventory#2659
erikzhang merged 3 commits intoneo-project:developfrom
shargon:optimize-01

Conversation

@shargon
Copy link
Member

@shargon shargon commented Feb 8, 2022

If knownHashes already contains the inventory, it should do nothing
Previous PR: #2630

@shargon shargon requested a review from erikzhang February 8, 2022 10:01
@shargon
Copy link
Member Author

shargon commented Feb 16, 2022

@erikzhang any feedback?

@erikzhang
Copy link
Member

Tested?

@shargon shargon requested a review from superboyiii March 31, 2022 09:24
@superboyiii superboyiii mentioned this pull request Apr 1, 2022
18 tasks
@superboyiii
Copy link
Member

Tested?

Tested OK. Works well.

@erikzhang erikzhang merged commit a02209a into neo-project:develop Apr 1, 2022
@shargon shargon deleted the optimize-01 branch April 4, 2022 08:50
shargon added a commit that referenced this pull request Apr 6, 2022
* Add ToJson overload (#2671)

* Add ToJson overload

* change

* Update src/neo/VM/Helper.cs

* Update src/neo/VM/Helper.cs

* Update src/neo/VM/Helper.cs

* Update src/neo/VM/Helper.cs

Co-authored-by: Jinghui Liao <jinghui@wayne.edu>

* Update src/neo/VM/Helper.cs

Co-authored-by: Jinghui Liao <jinghui@wayne.edu>

Co-authored-by: Shargon <shargon@gmail.com>
Co-authored-by: Jinghui Liao <jinghui@wayne.edu>

* Fix oom (#2665)

* Fix oom

* Revert reorder

* parameters order

Co-authored-by: Erik Zhang <erik@neo.org>

* Optimize inventory (#2659)

* add `murmur32` to crypto lib (#2604)

* 3.2.0

* fix

Co-authored-by: Shargon <shargon@gmail.com>
Co-authored-by: Jinghui Liao <jinghui@wayne.edu>
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

Successfully merging this pull request may close these issues.

4 participants