Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow for changing of system tree parent #71

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cvonelm
Copy link
Member

@cvonelm cvonelm commented Feb 11, 2025

This PR makes it possible to change the parent of a system_tree_node after it has already been set.

This is useful for lo2s, as I am in the process of rebuilding the Process insertion method.

Currently, we have to take much care to insert the process into the system tree only when we know its parent process, as we can not change it later.

This makes thinking about when in lo2s we already have the system_tree_node for a process, and when not, tedious.

With this change, we can just emplace a bare bones system tree node whenever we need it, as we can simply fill in missing information (such as the parent) later.

Makes it possible to update the parent if the information arrives lates

Also fix .clang-format using the one from lo2s
@cvonelm cvonelm force-pushed the allow_change_parent branch from a277dc6 to 882a13e Compare February 11, 2025 13:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant