Eure besten Quellcode Kommentare...

[himitsu]

Oder wie in den Indy, wo niemand es hinbekam ein Synchronisationsobjekt aufzuräumen, bzw. dass vor dessen "geplanter" Freigabe alle Abhängigen auch freigegeben sind,
und sie es daher sich sagen "Windows räumt am Ende eh auf", also lassen wir es einfach und damit keiner Denkt wir sind doof, verstecken registrieren wir diesen Speicher eben.

[Neutral General]

Delphi-Quellcode:

{$IFDEF MSWINDOWS}
procedure DLLShutdown; stdcall;
begin
  try
    if @DLLShutdownProc <> nil then
      DLLShutdownProc;
  finally
    // eat all exception.
  end;
end;
{$ENDIF MSWINDOWS}

[MaBuSE]

Zitat von Neutral General:

Delphi-Quellcode:

...
  finally
    // eat all exception.
  end;
...

Der ist gut

Aber sicherlich hat er zufällig (quasi aus versehen) die Exception in der aufrufenden Methode abgefangen

[freimatz]

Ausnahmsweise finde ich: Das ist wirklich gut. Wenn auch nur in diesem recht exotischen Fall und für eine Release-Version.
Bei denem Zitat hast du was wichtiges weggelassen so z.B. "DLLShutdown". Ich vermute es geht hier um die letzte Methode in einer DLL. Was soll denn die DLL mit der Exception machen. Vermutlich sind doch schon alle Systene runtergefahren die noch eine Möglichkeit der Behandlung bieten würde. (Vielleicht ginge noch ein Aufruf an madExcept oder so.) Die Exception an den zu überlassen, der die DLL entlädt hilft m.E. auch nicht weiter.

Aber ich pflichte den meisten bei, in den meisten Fällen ist das Schlucken aller Exception einfach nur

[Neutral General]

Die DLLShutdown-Methode die ich gepostet habe ist übrigens aus der System.pas. (10.3.2)
Genau so geschrieben und kommentiert.

@freimatz: Der Witz an der Sache ist, dass das finally die Exceptions nicht mal "eatet".
Das Try-Finally tut rein gar nichts. Was (laut Kommentar) gewollt wäre, ist ein try-except mit einem leeren except-Block.

[philipp.hofmann]

Ok, danke. Ich habe echt schon an mir gezweifelt, was das finally hier bitte schön "eaten" soll.

[freimatz]

Ah so jetzt ja. Sorry

[MaBuSE]

Zitat von Neutral General:

Die DLLShutdown-Methode die ich gepostet habe ist übrigens aus der System.pas. (10.3.2)
Genau so geschrieben und kommentiert.

Zitat von Neutral General:

@freimatz: Der Witz an der Sache ist, dass das finally die Exceptions nicht mal "eatet".
Das Try-Finally tut rein gar nichts. Was (laut Kommentar) gewollt wäre, ist ein try-except mit einem leeren except-Block.

Genau. Deshalb hatte ich ich ja noch geschrieben, dass es hoffentlich noch drüber abgefangen wird.

Zur Erklärung:
[EDIT]
Ich habe diesen (langen Text) in ein eigenes Thema verschoben, dort kann auch dazu diskutiert werden.

Kleiner Exkurs zu Exceptions (try finally/except):
https://www.delphipraxis.net/203329-...ly-except.html

[/EDIT]

[freimatz]

Das haste jetzt hoffentlich nicht wegen mir zusammengebröselt

[MaBuSE]

Zitat von freimatz:

Das haste jetzt hoffentlich nicht wegen mir zusammengebröselt

Äh, doch

Vielleicht findet es ja noch der ein oder andere interessant.