Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Mutable Error Message Pointer #131

Merged
merged 1 commit into from
Jan 4, 2024
Merged

Mutable Error Message Pointer #131

merged 1 commit into from
Jan 4, 2024

Conversation

threeal
Copy link
Owner

@threeal threeal commented Jan 4, 2024

This pull request fixes #129 by removing the const specifier from Error::msg_ptr to make that variable mutable, allowing copy assignment to the Error class.

@threeal threeal self-assigned this Jan 4, 2024
@threeal threeal added the fix label Jan 4, 2024
@threeal threeal added this to the Version 1.0.0 milestone Jan 4, 2024
@threeal threeal marked this pull request as ready for review January 4, 2024 05:07
Base automatically changed from conflicting-nil-name to main January 4, 2024 05:08
@threeal threeal force-pushed the mutable-message-pointer branch from c5af931 to 86595ac Compare January 4, 2024 05:09
@threeal threeal merged commit 645252d into main Jan 4, 2024
8 checks passed
@threeal threeal deleted the mutable-message-pointer branch January 4, 2024 05:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Constant Message Pointer Disables Copy Constructor
1 participant