Skip to content

Zap initial snapshots. Issue #113#122

Merged
haesbaert merged 2 commits into
mainfrom
no-snap
Feb 12, 2025
Merged

Zap initial snapshots. Issue #113#122
haesbaert merged 2 commits into
mainfrom
no-snap

Conversation

@haesbaert
Copy link
Copy Markdown
Collaborator

Zap initial snapshots. Issue #113
This removes the initial snapshot of processes that it was sent on queue bootup.
The user is now expected to use quark_queue_iter(3) to retrieve them.

This simplified the code quite a bit, and as we add more kind of events,
deciding what to send initially becomes silly. We provide decent iterators and
users rejoice.

Documentation follows on the next commit.

This removes the initial snapshot of processes that it was sent on queue bootup.
The user is now expected to use quark_queue_iter(3) to retrieve them.

This simplified the code quite a bit, and as we add more kind of events,
deciding what to send initially becomes silly. We provide decent iterators and
users rejoice.

Documentation follows on the next commit.
@haesbaert haesbaert requested a review from a team as a code owner February 11, 2025 07:03
Comment thread quark_process_iter.3
.Fn quark_process_iter_next "struct quark_process_iter *"
.Sh DESCRIPTION
Initialize and iterate over the internal cache of processes, it can be used on
boot up to get a list of all learned processes, or else.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or else what?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

or ELSE!!!!!!!
I should have written "or anything else"

@haesbaert haesbaert merged commit 1f94d2a into main Feb 12, 2025
@haesbaert haesbaert deleted the no-snap branch April 2, 2025 11:14
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.

2 participants