dotnet/winforms

DataGridView not dispose old elements

Open

#6859 aperta il 17 mar 2022

Vedi su GitHub
 (11 commenti) (0 reazioni) (0 assegnatari)C# (922 fork)batch import
area-controls-DataGridViewhelp wantedtenet-performance

Metriche repository

Star
 (4100 star)
Metriche merge PR
 (Merge medio 14g 22h) (56 PR mergiate in 30 g)

Descrizione

  • .NET Core Version: all up to 7.0 p2

  • Have you experienced this same bug with .NET Framework?: Yes

Problem description: As I mentioned in https://github.com/dotnet/winforms/issues/6858 DataGridView not dispose old data on DataSource change. This is lead to 2 problems:

  1. All this stuff are going to finalization query because we have unnecessary finalizer here #6858.
  2. If we have a ContextMenuStrip bound to Cell/Column/Row we will have a memory leak. DataGridView with 3 columns and ContextMenuStrip bound to 1000 rows, result of DS refresh: image

Expected behavior: Old data must be property disposed. Probably in DataGridView.RefreshColumnsAndRows() (in DataGridView.Methods.cs)

Minimal repro: DataGridViewLeak.zip

Guida contributor

Direzione di ricerca
Analizzare la logica di disposizione in `DataGridView.Methods.cs`, in particolare il metodo `RefreshColumnsAndRows()`. Rivedere il problema di riferimento #6858 riguardante i finalizzatori non necessari. Concentrarsi sul garantire che gli elementi vecchi vengano smaltiti correttamente quando cambia `DataSource`, come descritto nel report del bug e nei commenti.
Tech stack
csharp
Dominio
desktop
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C#IDisposable pattern
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70