forked from neo-project/neo
-
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
Preview5 to RC1 Comparison #6
Open
spencercorwin
wants to merge
63
commits into
preview5-untouched
Choose a base branch
from
rc1-untouched
base: preview5-untouched
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Verification optimizations * Fix UT * Add snapshot to MakeTransaction * Optimize * Update Transaction.cs * Fix * Fix UT * Revert Wallet change Co-authored-by: Erik Zhang <[email protected]>
* Add GetInstance * Add UT Co-authored-by: Erik Zhang <[email protected]>
* Add native contract: StdLib * format * Add CryptoLib * Add SCriptContainer.GetHashData * Return object instead of interop * Fix UT * Fix of fix Co-authored-by: Erik Zhang <[email protected]> Co-authored-by: Vitor Nazário Coelho <[email protected]>
* Clean EmitDynamicCall code * Add Safe callflag * Remove Safe * Update AssetDescriptor.cs Co-authored-by: Erik Zhang <[email protected]>
…2315) * Set max verification for state dependant * Update Transaction.cs * Prevent VerifyStateDependent if not enough gas in StaticIndependent * Check gas in wallet * Fix wallet condition * Update Transaction.cs * Remove wallet condition Co-authored-by: Erik Zhang <[email protected]>
Co-authored-by: Erik Zhang <[email protected]>
Co-authored-by: Shargon <[email protected]>
* Adjust native contract fee * Add default Oracle price * Charge the committee Co-authored-by: Shargon <[email protected]>
Co-authored-by: Shargon <[email protected]>
* Remove BitConverter * Optimize Co-authored-by: Erik Zhang <[email protected]>
Co-authored-by: Shargon <[email protected]>
Co-authored-by: Erik Zhang <[email protected]>
* Fix types in native contracts * format Co-authored-by: Shargon <[email protected]>
* Update VM * Update Sqlite * Update prices * Update price Co-authored-by: Erik Zhang <[email protected]>
* Add GetPendingWitnesses * GetSignatures * Remove clean signatures * Store single signatures in ContextItem * Clean code * readonly * Optimize * Change return type * Fix UT Co-authored-by: Erik Zhang <[email protected]>
* StorageItem: optimize binary representation After neo-project#2351 StorageItem is just a thin wrapper for some byte array in its essence so we can Use the same trick as was used for storage keys in neo-project#1566 and avoid additional wrapping/unwrapping with VarBytes encoding. * StorageItem: optimize out reader.BaseStream.Position It's always 0.
…method for use by blockchain toolkit (neo-project#2381)
* change prices * Revert change * Revert GetNextBlockValidators price * fix
) * SmartContract: rename hex in ContractParametersContext It's not really hex since neo-project#1199 and even before that it could've been named a bit differently. `item` is not an option because we already have `items` there, but we're signing things so probably it's OK to call this signed thing a `document`. * SmartContract: rename 'document' field to 'data' Seems to be better for this, thanks @shargon. Co-authored-by: Erik Zhang <[email protected]>
…roject#2396) * Make ContractManagement.Check available for blockchain toolkit * Move to Helper Co-authored-by: Erik Zhang <[email protected]>
* optimize transaction route * logic fix * logic fix * restore logic when Blockchain receives transaction
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.