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.

Sign up for GitHub

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

Jump to bottom

fix: panic salt live #61

Closed
wants to merge 2 commits into from
Closed

fix: panic salt live #61

wants to merge 2 commits into from

Conversation

kpetremann
Copy link
Owner

Fixes #55

@kpetremann
chore(live): style
3a40f7e
@kpetremann
fix(live): race condition
ca51235 
There was a race condition when adding/removing the eventList in Salt
Live.

The eventList is now managed through a delegate to enable adding a mutex
to avoid this.
@kpetremann kpetremann changed the title Fix panic salt live fix: panic salt live Dec 22, 2023
@kpetremann
Copy link
Owner Author

kpetremann commented Dec 28, 2023 •
edited
Loading

note: delegate.update is only called during browsing. It means that when filter is being changed, no items are processed at all.

@kpetremann
Copy link
Owner Author

not a good fix

@kpetremann kpetremann closed this Dec 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[salt-live] Invalid memory address
1 participant
@kpetremann