[CLOSED] Improvement of Destructor TComponent.Destroy;
lazarus / 64-FPC development

----------------------------------------------------------------------
lagprogramming · 2023-11-21 10:12

rtl/objpas/classes/compon.inc has Destructor TComponent.Destroy;
In order to exit a loop, the original code sets a zero value to
variable "I".
A patch has been attached that changes the line "I := 0;" with
"Break;", increasing code readability.

----------------------------------------------------------------------
AlexTP · 2023-11-21 11:29

Reported to
https://gitlab.com/freepascal.org/fpc/source/-/issues/40535

----------------------------------------------------------------------
AlexTP · 2023-11-22 06:09

Declined at FPC bugtracker.

----------------------------------------------------------------------
Leledumbo · 2023-11-24 04:49

Quote from: AlexTP on November 22, 2023, 07:09:22 amDeclined at FPC
bugtracker.
Expected. For those who live long enough with Pascal, break and
continue are indeed not inline with Pascal spirit. Most if not all of
my softwares don't use it and readability enhancement is subjective.