Skip to content

Fix IDisposable implementation in sealed classes#24719

Open
xtqqczze wants to merge 1 commit into
PowerShell:masterfrom
xtqqczze:sealed-dispose
Open

Fix IDisposable implementation in sealed classes#24719
xtqqczze wants to merge 1 commit into
PowerShell:masterfrom
xtqqczze:sealed-dispose

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

@xtqqczze xtqqczze commented Dec 29, 2024

  • Do not invoke GC.SuppressFinalize for types without a finalizer.
  • Remove redundant void Dispose(bool disposing) method for sealed types.

Best reviewed with whitespace hidden.

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Jan 2, 2025
@xtqqczze xtqqczze marked this pull request as ready for review January 17, 2025 13:15
@xtqqczze xtqqczze closed this Jan 17, 2025
@xtqqczze xtqqczze reopened this Jan 17, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot removed Stale Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept labels Jan 17, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Review - Needed The PR is being reviewed label Jan 25, 2025
@xtqqczze xtqqczze marked this pull request as draft February 1, 2025 18:20
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Feb 4, 2025
@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Feb 10, 2025
@xtqqczze xtqqczze force-pushed the sealed-dispose branch 3 times, most recently from b36cecc to e1aaa55 Compare February 10, 2025 13:20
@xtqqczze xtqqczze marked this pull request as ready for review February 11, 2025 14:43
@xtqqczze xtqqczze marked this pull request as draft August 28, 2025 14:47
@xtqqczze xtqqczze force-pushed the sealed-dispose branch 2 times, most recently from 08e2b65 to 92e1eea Compare August 31, 2025 10:31
@xtqqczze xtqqczze marked this pull request as ready for review August 31, 2025 10:34
@xtqqczze xtqqczze requested a review from TravisEz13 as a code owner August 31, 2025 10:34
@xtqqczze xtqqczze marked this pull request as draft February 27, 2026 19:49
@microsoft-github-policy-service microsoft-github-policy-service Bot added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 3, 2026
@xtqqczze

This comment was marked as resolved.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Mar 18, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept and removed Stale labels Mar 18, 2026
@xtqqczze xtqqczze marked this pull request as ready for review April 5, 2026 23:54
@microsoft-github-policy-service microsoft-github-policy-service Bot removed Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept Stale labels Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Review - Needed The PR is being reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant