Locks dienen im Gegenteil eher dem Performancegewinn (oder hier eben anderen Zwecken), weil man damit verhindert, dass jemand "von der Seite reingrätscht". Langsamer wird es meistens eher nicht.

Ich habe jetzt die Locks eingefügt. Ob es dadurch langsamer geworden ist, weiß ich nicht, aber ich habe dadurch den alten Code wieder durchgesehen und dabei eine völlig neue Erfahrung gemacht: Wenn man eine Bitmap freigibt, dann ist es immer gut, wenn man sie vorher erzeugt hat. Erst ein Create , dann ein später eingefügtes bedingtes Exit , danach ein unbedingtes Free - das ist nicht gut, auch ganz ohne multithreaded. Das Create hinter das Exit und weg sind die Exceptions. So einfach kann Programmieren sein.

Zitat von Benmik:

Wenn man eine Bitmap freigibt, dann ist es immer gut, wenn man sie vorher erzeugt hat.

Genauer gesagt: Wenn man sie zugewiesen hat, sprich mit nil initialisiert hat.

Solange die Variable entweder auf eine Bitmap verweist oder nil ist, ist es kein Problem ein Free aufzurufen, denn das prüft, ob Self <> nil ist.

Das Problem ist daher nicht das Free oder ein nicht immer aufgerufenes Create, sondern eine fehlende Initialisierung der Variablen.

Wobei normal der Compiler dann ja auch einen Tipp geben würde, wenn die Variable nicht initialisiert wurde.