Samsung / ONE-vscode

Visual Studio Code Extension of ONE compiler toolchain
Apache License 2.0
48 stars 50 forks source link

[OneExplorer] Review the necessity of NodeMap #1718

Open dayo09 opened 1 year ago

dayo09 commented 1 year ago

What?

NodeMap store path to node relationship. To fulfill this functionality, node insertion/deletion should be related to all the file creation/deletion.

Initially it was introduced to keep node status in spite of hide/show status. Let's check if it's really needed. Maybe nodes can be conditionally mapped to TreeItem, so that node map can be deprecated.


DRAFT: #1717

dayo09 commented 1 year ago

Usages

It maps filesystem change to node