-
Notifications
You must be signed in to change notification settings - Fork 77
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
interop: post-preview2 adjustment, part 3 #1221
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/neo-project/neo/blob/master/src/neo/SmartContract/ApplicationEngine.Storage.cs#L113
StorageFlags
is defined as a enum in C#, so any odd flag denotes const, not only 1
.
Implementation of PutEx
and docs should be fixed. Pre-defined constant flags will increase readability.
f0b68aa
to
6e44499
Compare
Added StorageFlags, but not sure it's the right place for them. |
Codecov Report
@@ Coverage Diff @@
## master #1221 +/- ##
==========================================
+ Coverage 66.92% 66.94% +0.02%
==========================================
Files 199 199
Lines 16991 16997 +6
==========================================
+ Hits 11371 11379 +8
+ Misses 5008 5005 -3
- Partials 612 613 +1
Continue to review full report at Codecov.
|
@@ -35,6 +35,7 @@ var syscalls = map[string]map[string]Syscall{ | |||
"ECDSASecp256k1CheckMultisig": {"Neo.Crypto.CheckMultisigWithECDsaSecp256k1", false}, | |||
"ECDsaSecp256r1Verify": {"Neo.Crypto.VerifyWithECDsaSecp256r1", false}, | |||
"ECDSASecp256r1CheckMultisig": {"Neo.Crypto.CheckMultisigWithECDsaSecp256r1", false}, | |||
"RIPEMD160": {"Neo.Crypto.RIPEMD160", false}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a commit description that this commit closes 1193. Isn't it enough or I should create a separate PR for this?
The third part of #1055.
Neo.Crypto.RIPEMD160 was added. (closes Implement Neo.Crypto.RIPEMD160 syscall #1193)
Were fixed:
System.Storage.GetContext
System.Storage.GetReadOnlyContext
System.Storage.Put
System.Storage.PutEx
Neo.Native.Deploy
System.Storage.Delete
System.Storage.Find
System.Storage.Get
System.Storage.AsReadOnly
Neo.Crypto.VerifyWithECDsaSecp256r1
Neo.Crypto.VerifyWithECDsaSecp256k1
Neo.Crypto.CheckMultisigWithECDsaSecp256r1
Neo.Crypto.CheckMultisigWithECDsaSecp256k1
Neo.Crypto.SHA256
For the rest of them there are separate issues, so that' the last part of 1055.