Skip to content

Conversation

@anatawa12
Copy link
Contributor

@anatawa12 anatawa12 commented Dec 5, 2024

Fix #16

I don't know if this is the best API, but works well with my program.

@anatawa12
Copy link
Contributor Author

I have fixed several error from example code.

@nptr
Copy link
Member

nptr commented Dec 6, 2024

Thanks a lot! Looks good! I'll check it out and run it myself this weekend, then merge it :)
If time allows I might push it to crates.io also. Last revision was some time ago.

/// });
///
/// # }
/// ```
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Suggested change
/// ```
/// ```
///
/// **Note**
/// The data held by this struct will be leaked if
/// libui is uninitialized when this struct is dropped.

It might be better to note there is possibility for leak data

@nptr nptr merged commit 2e5625f into libui-rs:development Dec 8, 2024
3 checks passed
@anatawa12 anatawa12 deleted the queue-main-from-other-thread branch December 8, 2024 13:23
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.

How can I send data to UI from another thread?

2 participants