dotnet/winforms

Don't force creation of `Handle` in `OnHandleCreated`

Open

#2235 aperta il 29 ott 2019

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)C# (922 fork)batch import
:beetle: bughelp wantedwaiting-on-team

Metriche repository

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

Descrizione

See https://github.com/dotnet/winforms/pull/2141

The base class Control.OnHandleCreated checks for IsHandleCreated and does nothing if the handle isn't created.

In other sub-classes, we do not check if (IsHandleCreated) before accessing this.Handle which forces creation. This leads to strange recursive behaviour as OnHandleCreated is called again.

My opinion is that we should fix this oversight in base classes to ensure that they don't create the Handle in OnHandleCreated if a Handle doesn't exist

Guida contributor

Direzione di ricerca
Ispeziona il PR di riferimento (dotnet/winforms#2141) e rivedi le sottoclassi di Control che sovrascrivono OnHandleCreated per identificare i casi in cui Handle viene accesso inutilmente. Assicurati di controllare IsHandleCreated prima di accedere a Handle.
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#.NETWindows Forms
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
55