Skip to content

fix: infix operation handling#8

Merged
brech1 merged 7 commits intomainfrom
fix/infix-op
Jan 10, 2024
Merged

fix: infix operation handling#8
brech1 merged 7 commits intomainfrom
fix/infix-op

Conversation

@brech1
Copy link
Copy Markdown
Contributor

@brech1 brech1 commented Jan 4, 2024

Description

This PR aims to fix the multi term infix operations processing. We're now able to process these operations and create the corresponding gates if necessary.

Changes

  • Remove every tuple return and replace with Result
  • Implement Result return on functions without return, in order to remove unwraps and improve error handling.
  • Update execute_expression to return a value of the processed expression.
  • Update execute_infix_op to run the operation and return the result.
  • Fix substitution handling in execute_statement.
  • Remove unused variable name argument from execute_expression and traverse_expression.
  • Implement ProgramError.
  • Update runtime logs.
  • Implement declare_auto_signal in the runtime.

@brech1 brech1 marked this pull request as ready for review January 9, 2024 21:51
@brech1 brech1 merged commit 50a40c1 into main Jan 10, 2024
@brech1 brech1 deleted the fix/infix-op branch January 11, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant