Why is this an issue?

Manually computing the index of the last element in a TList descendant is less clear and less terse than using TList.Last. The Last method is equivalent in functionality, and it's inlined so there shouldn't be any performance difference.

How to fix it

Use TList.Last instead of indexing with Count - 1:

procedure PrintMostRecent(MyList: TList<Integer>);
begin
  WriteLn(MyList[MyList.Count - 1]);
end;

procedure PrintMostRecent(MyList: TList<Integer>);
begin
  WriteLn(MyList.Last);
end;

• RAD Studio documentation: System.Generics.Collections.TList.Last
• RAD Studio documentation: System.Classes.TList.Last