blikblum/VirtualTreeView-Lazarus

Infinite loop in GetPreviousVisibleNoInit()

Closed this issue 3 years ago · 1 comments

salvadorbs commented 3 years ago

From Lazarus Issues (https://bugs.freepascal.org/view.php?id=38836):

There is an infinite loop in GetPreviousVisibleNoInit() when toChildrenAbove is used.

Patches https://github.com/graemeg/lazarus/commit/51cdf886da59016ad6f43dcc5ab493ad824ef89f and https://github.com/graemeg/lazarus/commit/a42d8ff76403e6545fb258549bc5b59cf8209525

Test Project by Martok

In the next few days, I will open a PR to apply the changes.

salvadorbs commented 3 years ago

Ops, I didn't see the wp's PR. Very good! Thanks!