Handle count erhöht sich permanent

[himitsu]

Zitat:

Der ThreadTask wird ausserhalb erstellt

Und wo ist dann die fCriticalSection, beim Durchlaufen und vorallem Löschen der Items?

Wie gesagt, wenn du im Constructor und Execute deinen Code mal auskommentierst, bleiben dann die Handle gleich?




Bezüglich deinem komischen OnTerminate-Konstrukt:

Delphi-Quellcode:

protected
  procedure DoTerminate; override;

procedure TThreadTask.DoTerminate;
begin
  inherited;
  ...
end;

// bzw., wenn der Code im Hauptthread ausgeführt werden soll (OnTerminate wird ja auch synchronisiert)

procedure TThreadTask.DoTerminate;
begin
  inherited;
  Synchronize(procedure
    begin
      ...
    end);
end;

[Klaus01]

Zitat von himitsu:

Zitat:

Der ThreadTask wird ausserhalb erstellt

Und wo ist dann die fCriticalSection, beim Durchlaufen und vorallem Löschen der Items?

Wie gesagt, wenn du im Constructor und Execute deinen Code mal auskommentierst, bleiben dann die Handle gleich?

Brauche ich beim Löschen eine CriticalSection?
fRunningTasks ist eine Liste die nur im Threadpool existiert.

Die TaskList ist eine Liste die von aussen befüllt wird.
Dort sind "geparkten" ThreadTasks enthalten.

Aus der TaskListe werden die ThreadTasks in die fRunningTaskliste verschoben und anschließend gestartet.
Gut beim Verschieben aus der TaskList in die fRunningTasks Liste, da könnte eine criticalSection fehlen.

Den onTerminate Konstrukt werde ich mir nochmals anschauen.

Wenn der Excute-code leer ist erhöht sich die Anzahl der Handles auch.
Auch wenn der Constructor leer ist, erhöht sich die Anzahl der Handles.

Grüße
Klaus

[Blup]

destructor TThreadTask.Destroy;

Fehlt da nicht ein "inherited"?

[himitsu]

Zitat von Blup:

Fehlt da nicht ein "inherited"?

Ja.

[Klaus01]

sodele..

ich habe nun das komische onTerminate Konstrukt abgeändert
und das fehlende inherited in der Destroy-Methode hinzugefügt.

.. und es schaut sehr viel besser aus.
Es wird die Anzahl der Handles nur um 1 pro Durchlauf erhöht.
.. das werde ich dann auch noch finden.

Danke für die Hilfe.

Grüße
Klaus