Skip to content

Cleanup for carrot_core#5

Merged
jeffro256 merged 3 commits intojeffro256:carrot_corefrom
UkoeHB:carrot_core_cleanup
Apr 8, 2026
Merged

Cleanup for carrot_core#5
jeffro256 merged 3 commits intojeffro256:carrot_corefrom
UkoeHB:carrot_core_cleanup

Conversation

@UkoeHB
Copy link
Copy Markdown

@UkoeHB UkoeHB commented Apr 7, 2026

Mostly comments and a couple function renames for consistency/accuracy. Removed references to Seraphis.

Will approve monero-project#9559 once this is merged.

@UkoeHB UkoeHB force-pushed the carrot_core_cleanup branch from 7c85294 to 78363ed Compare April 7, 2026 23:34
@UkoeHB UkoeHB force-pushed the carrot_core_cleanup branch from 78363ed to 3528738 Compare April 7, 2026 23:50
@UkoeHB
Copy link
Copy Markdown
Author

UkoeHB commented Apr 8, 2026

CI error with Arch Linux finding Boost, seems unrelated.

Copy link
Copy Markdown
Owner

@jeffro256 jeffro256 left a comment

Choose a reason for hiding this comment

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

Thank you so much for reviewing!! Largely agree with the changes, just have a few comments.

Comment thread src/carrot_core/carrot_enote_types.cpp
Comment thread src/carrot_core/address_utils.cpp Outdated
Comment thread src/carrot_core/address_utils.cpp
Comment thread src/carrot_core/address_utils.cpp Outdated
Comment thread src/carrot_core/account_secrets.cpp Outdated
Comment thread src/carrot_core/core_types.h Outdated
Comment thread src/carrot_core/destination.h
Comment thread src/carrot_core/enote_utils.cpp Outdated
Comment thread src/carrot_core/core_types.h Outdated
Co-authored-by: Jeffro <jeffro256@tutanota.com>
@jeffro256 jeffro256 merged commit f7707da into jeffro256:carrot_core Apr 8, 2026
@UkoeHB UkoeHB deleted the carrot_core_cleanup branch April 8, 2026 22:13
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