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

Allow not null implementation of IDisposable in CEs #395

Conversation

gdziadkiewicz
Copy link
Collaborator

@gdziadkiewicz gdziadkiewicz commented Jul 9, 2020

Current implementation backing use! in most of the CEs for the IDisposable implementing type to be null allowing. This is an artificial requirement which limits the usefulness of the code.

This PR amends the implementations to allow type for which null is not a valid value.

@gdziadkiewicz
Copy link
Collaborator Author

Remove WIP commit.

FSharpx.Extras.sln Outdated Show resolved Hide resolved
@gdziadkiewicz gdziadkiewicz force-pushed the feature/Allow_not_null_implementation_of_IDisposable_in_CEs branch 2 times, most recently from 95569f6 to 96bcd0f Compare July 11, 2020 16:20
@gdziadkiewicz gdziadkiewicz marked this pull request as ready for review July 11, 2020 16:21
@gdziadkiewicz gdziadkiewicz force-pushed the feature/Allow_not_null_implementation_of_IDisposable_in_CEs branch from 96bcd0f to 77a61c5 Compare July 11, 2020 16:47
@gdziadkiewicz gdziadkiewicz merged commit d4308dc into fsprojects:master Jul 11, 2020
@gdziadkiewicz gdziadkiewicz deleted the feature/Allow_not_null_implementation_of_IDisposable_in_CEs branch July 11, 2020 17: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.

1 participant