Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Move standard System.Crypto.CheckSig invocation script length to a designated constant #3797

Open
AliceInHunterland opened this issue Jan 29, 2025 · 0 comments
Open

Move standard System.Crypto.CheckSig invocation script length to a designated constant #3797

AliceInHunterland opened this issue Jan 29, 2025 · 0 comments
Labels
enhancement Improving existing functionality I4 No visible changes S4 Routine U4 Nothing urgent
Milestone
v0.109.0

Comments

@AliceInHunterland
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Sometimes we do have just 66 without any description.
#3789 (comment) Example of implementation 6ed2f79

Describe the solution you'd like

Create a constant for this.

Describe alternatives you've considered

no

Additional context
@AliceInHunterland AliceInHunterland added enhancement Improving existing functionality I2 Regular impact I4 No visible changes S4 Routine U4 Nothing urgent labels Jan 29, 2025
@AliceInHunterland AliceInHunterland mentioned this issue Jan 29, 2025
Merged
@AnnaShaleva AnnaShaleva removed the I2 Regular impact label Jan 30, 2025
@AnnaShaleva AnnaShaleva added this to the v0.109.0 milestone Jan 30, 2025
@AnnaShaleva AnnaShaleva changed the title Magic number in InvocationScript Move standard System.Crypto.CheckSig invocation script length to a designated constant Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Improving existing functionality I4 No visible changes S4 Routine U4 Nothing urgent
Projects
None yet
Milestone
v0.109.0
Development

No branches or pull requests
2 participants
@AnnaShaleva @AliceInHunterland