Delphi Bugs mit Lösungen...

[Uwe Raabe]

Zitat von Mavarik:

Delphi-Quellcode:

  if Assigned(FMethods) then // 
    FMethods.Free;           // Fehlte

Das if Assigned kannst du dir sparen, das macht das Free selbst. Unter ARC wird das vom Compiler sowieso durch eine nil-Zuweisung ersetzt, was die Abfrage ebenso obsolet macht.

[Mavarik]

Zitat von Uwe Raabe:

Zitat von Mavarik:

Delphi-Quellcode:

  if Assigned(FMethods) then // 
    FMethods.Free;           // Fehlte

Das if Assigned kannst du dir sparen, das macht das Free selbst. Unter ARC wird das vom Compiler sowieso durch eine nil-Zuweisung ersetzt, was die Abfrage ebenso obsolet macht.

Auf eine Methode eine Klasse zuzugreifen ohne zu testen, ob die Klasse Assigned ist - widerstrebt mir einfach...

[Darlo]

Zitat von Mavarik:

Delphi-Quellcode:

  if Assigned(FMethods) then // 
    FMethods.Free;           // Fehlte

Sollte das nicht besser so heißen?

Delphi-Quellcode:

  if Assigned(FMethods) then 
  {$IFDEF AUTOREFCOUNT}
    FMethods.DisposeOf;
  {$ELSE}
    FMethods.Free;

Warum wird im Free eigentlich nicht geprüft ob ARC und dann DisposeOf aufgerufen? Würde ne Menge an Code sparen.

[Uwe Raabe]

Zitat von Darlo:

Warum wird im Free eigentlich nicht geprüft ob ARC und dann DisposeOf aufgerufen? Würde ne Menge an Code sparen.

Weil Free unter ARC eben gar nicht aufgerufen wird! Der Compiler ersetzt das einfach durch eine nil-Zuweisung.

[Darlo]

Dann könnte der Compiler das doch durch disposeof ersetzen

[Fritzew]

Zitat von Darlo:

Dann könnte der Compiler das doch durch disposeof ersetzen

Und für was sollte das dann gut sein?
Disposeof wird nur benötigt wenn da mehrere Referenzen auf das selbe Object sind, und Ressourcen freigegeben werden müssen.
Ansonsten besteht keinerlei Grund das einzusetzen.

[Uwe Raabe]

Zitat von Darlo:

Dann könnte der Compiler das doch durch disposeof ersetzen

Nicht unbedingt, denn DisposeOf macht etwas anders als eine nil-Zuweisung.