Objekt auf Übergabe prüfen.

[DeddyH]

Noch richtiger wäre es IMO mit
Assert(Assigned(sl)); am Funktionsanfang.

[Sir Rufo]

Zitat von DeddyH:

Noch richtiger wäre es IMO mit
Assert(Assigned(sl)); am Funktionsanfang.

Nein, das ist in diesem Fall nicht notwendig, denn dann wird in der Test-Prozedur auch gleich zu Anfang eine Exception geworfen.
Problematisch sind ja immer nur die Fälle, wo die Exception nicht da auftritt, wo diese eigentlich verursacht wurde, denn dann kann man sich einen Wolf suchen.

Also wie in der Test-Prozedur, mit dem TStringList.OwnsObjects.