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

Fix etl::expected assignment from etl::unexpected #733

Conversation

Vitmark97
Copy link
Contributor

Assignment operator of etl::expected form etl::unexpected attempts to save etl::unexpected to storage instead of the contained error.

@semanticdiff-com
Copy link

View changes with SemanticDiff

@jwellbelove jwellbelove changed the base branch from master to pull-request/#733-Fix-etl--expected-assignment-from-etl--unexpected July 18, 2023 05:40
@jwellbelove jwellbelove merged commit 9cd491e into ETLCPP:pull-request/#733-Fix-etl--expected-assignment-from-etl--unexpected Jul 18, 2023
58 checks passed
jwellbelove added a commit that referenced this pull request Jul 18, 2023
jwellbelove added a commit that referenced this pull request Jul 18, 2023
Added unit tests for changes
Added missing etl::forward in etl::expected::emplace
jwellbelove added a commit that referenced this pull request Jul 18, 2023
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