Skip to content

Use ObjectDisposedException throw helper#3909

Merged
jeremydmiller merged 1 commit intoJasperFx:masterfrom
HakamFostok:object-disposed-exception
Aug 19, 2025
Merged

Use ObjectDisposedException throw helper#3909
jeremydmiller merged 1 commit intoJasperFx:masterfrom
HakamFostok:object-disposed-exception

Conversation

@HakamFostok
Copy link
Contributor

@HakamFostok HakamFostok commented Aug 11, 2025

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1513

It seems that the second parameter needs to the instance of the object (by passing this keyword or the Type of the containing class), and the message of the exception can not be changed. But I did not want to make a breaking change, even if it was very small, so I passed the string as it is without any change

https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/quality-rules/ca1513

It seems that the second parameter need to be the full name type of the class, and the message of the exception could not be changed. But I did not want to make breaking change, even if it very small, so I passed the string as it is without any change
@jeremydmiller jeremydmiller merged commit 8625277 into JasperFx:master Aug 19, 2025
3 checks passed
@HakamFostok HakamFostok deleted the object-disposed-exception branch August 21, 2025 09:28
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