wish: little changes for legacy projects

9 views
Skip to first unread message

Arioch The

unread,
Sep 13, 2016, 10:19:47 AM9/13/16
to Virtual Treeview
1. TVTGetImageEvent and TVTGetImageEventEx were changd: last paramaeter form Integer was changed to TImageIndex

However many units do not use VCL.ImgList but onul VTV itself

Wish: VirtualTrees.pas had this type reintroduced in a typical Delphi RTL fashion: type TImageIndex = Vcl.ImgList.TImageIndex;


2. In VTV 5.x VirtualTreeView.Root.Parent = nil
This was utilized in those typical bottom-up rocessing routines

while Node.Parent <> nil do begin
  ....
  Node := Node.Parent;
end;

However in VTV 6 there seems some garbage there and the loop foes astray

If possible, it would be nice for that to be brought back.
Both for legacy code and for using typical well recognized "linked list" loops instead of customary and untypical "while Node <> Node.TreeView.Root do"

Joachim Marder

unread,
Sep 13, 2016, 1:59:28 PM9/13/16
to virtual-...@googlegroups.com
Hi.

1. has already been done in GitHub master and will be part of the next version 6.4.

2. I have no clue when and why this has changed. I looked into InitRootNode() of V5.5 and GitHub master and bot set Parent := Pointer(Self);
I any case it would be good if you could open an issue for future reference and for discussion.

Joachim


--
You received this message because you are subscribed to the Google Groups "Virtual Treeview" group.
To unsubscribe from this group and stop receiving emails from it, send an email to virtual-treeview+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply all
Reply to author
Forward
0 new messages