CommunityToolkit / CommunityToolkit/Windows
Clean up suppression of CA1063 and implement IDisposable correctly
- Lenguaje dominante
- C#
- Estrellas
- 1.1k
- Forks
- 166
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
From https://github.com/CommunityToolkit/Tooling-Windows-Submodule/issues/63
One or more components in this repository are suppressing CA1063. To avoid memory leaks in certain edge cases, we should be implementing the full `IDisposable` pattern and not suppressing this warning.
[Affected components](https://github.com/search?q=repo%3ACommunityToolkit%2FWindows+disable+CA1063&type=code):
- [CameraHelper](https://github.com/CommunityToolkit/Windows/blob/744e6aba53329b3f04980237ca4d19eae3fbd5c9/components/Helpers/src/CameraHelper/CameraHelper.cs#L16)
- [AdvancedCollectionView](https://github.com/CommunityToolkit/Windows/blob/744e6aba53329b3f04980237ca4d19eae3fbd5c9/components/AdvancedCollectionView/src/AdvancedCollectionView.Defer.cs#L24)
- [ExpressionNode](https://github.com/CommunityToolkit/Windows/blob/744e6aba53329b3f04980237ca4d19eae3fbd5c9/components/Animations/src/Expressions/ExpressionNodes/ExpressionNode.cs#L18) (from Animations package)
Guía de contribución
Línea de trabajo
Inspecciona CameraHelper.cs, AdvancedCollectionView.Defer.cs y ExpressionNode.cs en las ubicaciones enlazadas y, a continuación, rastrea cómo cada componente posee recursos que deben liberarse. Verifica primero las supresiones de CA1063 y el comportamiento de liberación existente; se considera terminado cuando los componentes afectados implementan el patrón completo de IDisposable sin suprimir CA1063.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- csharp
- Área
- tooling
- Tipo de issue
- Refactorización
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100