Supports(..) liefert Referenz welche AV auslöst

[himitsu]

Zitat:

_AddRef(); // Damit mich das Supports(..) nicht abräumt

Da ist aber jemand selber Schuld!

Entweder man benutzt nur Interface-Referenzen oder nur Objekt-Referenzen.
Nur z.B. bei TComponent-Nachfahren kann man Beides benutzen, da dort die Referenzzählung deaktiviert wurde.

[Sir Rufo]

@himitsu

TInterfacedPersistent

Und man kann sich auch eine eigene Interfaced-Klasse bauen, die keine Referenzzählung hat.

[himitsu]

Sir Rufo:
Drum "z.B." ... halt Alles, wo die Referenzzählung nicht für die Freigabe benutzt wird.

@mjustin:
Nein, das Leak ist her nur, weil TComponent/TInterfacedObject nicht über die Referenzzählung freigegeben wird, sondern ausschließlich über das Free der Objektinstanz.
Das wurde absichtlich so gemacht, damit die Instanzen nur von der VCL verwaltet werden und man dennoch Interfaces benutzen kann (nur für Funktionszugriffe und nicht für die Freigabe).

Also bist DU an den Leak Schuld, denn du versuchst ein Interface über die Referenzzzählung freizugeben, welches darüber nicht freigegeben wird.

[mjustin]

Zitat von himitsu:

Nein, das Leak ist her nur, weil TComponent/TInterfacedObject nicht über die Referenzzählung freigegeben wird, sondern ausschließlich über das Free der Objektinstanz.

Wird hier "Compilermagie" anstelle Referenzzählung verwendet? (Nach der Antwort Nummer eins auf Stackoverflow wird Referenzzählung verwendet, diese aber scheitert da die _Release Methode bei TComponent anders als bei TInterfacedObject implementiert ist, um die Lebenszeit über Objektbesitzerschaft zu steuern). Das Free führt daher - wenn ich es richtig verstehe - in beiden Fällen zu einem Dekrementieren des Referenzzählers und dem dadurch aufgerufenen _Release.

[himitsu]

Zitat:

Delphi-Quellcode:

function TComponent._Release: Integer;
begin
  if FVCLComObject = nil then
    Result := -1   // -1 indicates no reference counting is taking place
  else
    Result := IVCLComObject(FVCLComObject)._Release;
end;

Nein, TComponent wird niemals über eine Interface-Referenz freigegeben.
TComponent wird vorwiegend innerhalb der VCL verwendet und Diese gibt ausschließlich ihre Instanzen über Free frei,
da es z.B. keine Weak-Referenzen für Objekte gibt, welche bei Freigabe auf nil gesetzt werden.

Es wird maximal das in TComponent gekapselte Interface freigegeben, wenn RefCount auf 0 fällt.
z.B. bei sowas wie TXMLDokument TVCLAutoObject oder TActiveFormControl (spezielles TActiveXControl), wo die eigentliche Funktion in ein Interface weitergeleitet wird. (siehe IVCLComObject)

[mjustin]

Zitat von himitsu:

Nur z.B. bei TComponent-Nachfahren kann man Beides benutzen, da dort die Referenzzählung deaktiviert wurde.

Benutzen schon, aber eine Komponente einfach wie im folgenden Beispiel um ein Interface zu erweitern geht nur um den Preis von Memory Leaks. Der folgende Code erzeugt ein Leak einer Instanz von TMyImplementation:

Delphi-Quellcode:

program LeakTest;

uses
  Classes;

type
  MyInterface = interface
  end;

  TMyImplementation = class(TComponent, MyInterface)
  end;

  TMyContainer = class(TObject)
  private
    FInt: MyInterface;
  public
    property Impl: MyInterface read FInt write FInt;
  end;

var
  C: TMyContainer;
begin
  ReportMemoryLeaksOnShutdown := True;

  C := TMyContainer.Create;
  C.Impl := TMyImplementation.Create(nil);
  C.Free;
end.

Das Memory Leak ergibt sich nur bei TComponent als Basisklasse. Mit TInterfacedObject als Basisklasse wird das über Interface verwaltete Feld FInt beim Destroy mit freigegeben, und kein Leak bleibt zurück.

Unter Stackoverflow kann man den Hintergrund dieses Verhaltens detailliert nachlesen.
Why do interface implementations based on TComponent leak memory?