🚀 go-pugleaf

Sven Callewaert wrote:

> Het antwoord op jouw geval staat op regel 886 van wincore.cpp. Feit is dat
> je een CWnd (ie. CChatDlg) uitwisselt met een andere thread en dat is niet
> threadsafe. Als je vanuit beide threads eenzelfde memberobject verandert kan
> dat
> onvoorspelbaar gedrag of een crash veroorzaken. Dat kan opgelost worden met
> locking (via mutexen en zo), maar dat is reeds advanced programming.

Het probleem hier is geen synchronisatie, maar het feit
dat MFC objecten thread-local zijn.  Elke thread heeft
een verzameling dictionaries, die bijhouden welke
CWnd er bij een HANDLE hoort, bv.  Als je nu van de
ene thread naar de andere een CChatDlg* doorstuurt,
krijg je die error omdat MFC hier een (child van een)
CWnd* te zien krijgt die niet in de dictionary van die
thread voorkomt.  Op zich is dat geen ramp want alles
bevindt zich nog steeds in hetzelfde proces, en dus
zijn memory en kernelhandles valid.  Het punt is alleen
dat als deze CChatDlg* nu zelf ook weer MFC gaat
gebruiken, MFC totaal niet meer gaat kunnen
volgen.  Bv, stel dat je deze CChatDlg hier zou
closen, dan gaat de destructor proberen de binding
HANDLE -> CWnd uit die dictionary te halen, wat ook
weer erg fout gaat wegens onvindbaar.  Idem bij
gebruik van CFont's en dergelijk die de dialog
nog in bezit heeft.

.... kortom : telkens als je tussen threads moet
praten, moet je niet-MFC objecten doorgeven.

Een klein(er) object doorgeven garandeert geen
thread-safety, dus dat moet je ook nog voorzien.

Verder zijn er ook nog issues met de message pump,
en wat MFC daar allemaal bovenop hackt, vooral als
het om dialogs gaat.

Dit alles maakt dat je de user interface echt wel
in 1 thread wil houden, tenzij je er eens ferm wil
invliegen voor een paar maanden :)  Multithreading
op zichzelf, plus dan nog met win32, plus dan nog
eens onder MFC, is Niet Om Te Lachen; dus je zou
best kijken of je dat hele idee niet kan afvoeren,
of anders, of je het kan inpassen in een "prefab"
scenario met worker threads en 1 UI thread (zie
google).

Bert