-
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
Fix pickitem and vm cli 2.x #967
Conversation
It was setting a (wrong) breakpoint and couldn't then break out of it.
bafdb91 change was wrong (probably brought from neo-vm 3.0 at the state at which it existed back then), neo-vm 2.x doesn't allow PICKITEM for arbitrary types.
Codecov Report
@@ Coverage Diff @@
## master-2.x #967 +/- ##
===========================================
Coverage 68.21% 68.21%
===========================================
Files 144 144
Lines 14185 14181 -4
===========================================
- Hits 9676 9674 -2
+ Misses 4057 4055 -2
Partials 452 452
Continue to review full report at Codecov.
|
return nil, err | ||
} | ||
item = vm.NewBigIntegerItem(val) | ||
case stringType: |
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.
mabe some sort of hex will also be useful
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 is a lot of room for improvement there, add an issue for it.
Problem
#965.
Solution
Fix it and improve vm cli a bit.