Allow not null implementation of IDisposable in CEs by gdziadkiewicz · Pull Request #395 · fsprojects/FSharpx.Extras · GitHub
Skip to content

Allow not null implementation of IDisposable in CEs#395

Merged
gdziadkiewicz merged 2 commits into
fsprojects:masterfrom
gdziadkiewicz:feature/Allow_not_null_implementation_of_IDisposable_in_CEs
Jul 11, 2020
Merged

Allow not null implementation of IDisposable in CEs#395
gdziadkiewicz merged 2 commits into
fsprojects:masterfrom
gdziadkiewicz:feature/Allow_not_null_implementation_of_IDisposable_in_CEs

Conversation

@gdziadkiewicz

@gdziadkiewicz gdziadkiewicz commented Jul 9, 2020

Copy link
Copy Markdown
Collaborator

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
Copy Markdown
Collaborator Author

Comment thread FSharpx.Extras.sln
@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