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.

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

Small Lem refactors #673

Closed
wants to merge 2 commits into from
Closed

Conversation

huitseeker
Copy link
Member

Small changes while reading (mostly) LEM code.

- Enhanced error handling in `verify` function within `clutch/src/lib.rs` to improve program stability.
- Updated import specifications across numerous files for clarity:, these include changes to `backend.rs`, `circuit.rs`, `path.rs`, `eval.rs`, `macros.rs`, and `pointers.rs`.
- Modified and refactored code for improved error handling, examples include changes to `interpreter.rs`, `mod.rs`, `var_map.rs`, `writer.rs`, `store.rs`.
- Included implementation of Debug traits in `var_map.rs` and `zstore.rs`.
- Enhanced handling of invalid u16 values in `Tag` implementation
- Implemented several lint checks in `.cargo/config` to ensure code quality.
- Refined match syntax in the `num_paths_taken` method on `path.rs` and `fetch_symbol_path`, `read_maybe_meta`, `hash_ptr` on `store.rs` for better performance and error handling.
@huitseeker huitseeker requested review from a team as code owners September 15, 2023 12:52
Copy link
Member

@arthurpaulino arthurpaulino left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@huitseeker
Copy link
Member Author

Will integrate and merge this in #660

@huitseeker huitseeker closed this Sep 17, 2023
auto-merge was automatically disabled September 17, 2023 18:32

Pull request was closed

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.

2 participants