Skip to content

feat: Refactor errors using macro #2061

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

leongross
Copy link
Member

@leongross leongross commented Mar 31, 2025

Refactor error code definitions based on the macro introduced in eee1a37. This PR can be considered a backport of #1992 for version 1.x

@leongross leongross requested a review from Copilot March 31, 2025 14:45
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

@leongross leongross marked this pull request as ready for review March 31, 2025 15:02
Copy link
Collaborator

@jhand2 jhand2 left a comment

Choose a reason for hiding this comment

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

Looks like this modifies ROM. Can you double check that none of the error codes were modified?

@leongross leongross force-pushed the leongross/unique-error-macros-1.x branch from 382eeb9 to 90072bd Compare March 31, 2025 16:20
Copilot

This comment was marked as duplicate.

@leongross
Copy link
Member Author

@jhand2

@leongross
Copy link
Member Author

@jhand2 the ROM is now unmodified, all valaues match.

Refactor error code definitions based on the macro introduced in
eee1a37.

Signed-off-by: leongross <[email protected]>
@leongross leongross force-pushed the leongross/unique-error-macros-1.x branch from 90072bd to a79383c Compare April 10, 2025 06:40
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