Skip to content

Commit

Permalink
fix: included argshash computation in public call_interfaces and clea…
Browse files Browse the repository at this point in the history
…nup (#7354)

During the development of the defunct
#6751 I realized we
weren't constraining the args_hash computation in public interfaces, but
promptly forgot during the offsite craze.

This PR fixes that and cleanups the interface computation for
readability. I would have loved to include the checks in the call
interface itself instead of having it in the less clear macros, but we
remove an `if` statement if the function takes no args, which in circuit
land is very good.
  • Loading branch information
Thunkar authored Jul 5, 2024
1 parent 533c937 commit 13e9b94
Show file tree
Hide file tree
Showing 2 changed files with 80 additions and 233 deletions.
Loading

0 comments on commit 13e9b94

Please sign in to comment.