Skip to content

refactor(runtime): timer - #995

Open
George-Miao wants to merge 2 commits into
compio-rs:masterfrom
George-Miao:refactor/runtime/timer
Open

refactor(runtime): timer#995
George-Miao wants to merge 2 commits into
compio-rs:masterfrom
George-Miao:refactor/runtime/timer

Conversation

@George-Miao

Copy link
Copy Markdown
Member

No description provided.

@github-actions github-actions Bot added enhancement New feature or request package: runtime Related to compio-runtime labels Aug 3, 2026
@George-Miao
George-Miao force-pushed the refactor/runtime/timer branch 4 times, most recently from 3b31b1f to 882f1f7 Compare August 3, 2026 03:27
@George-Miao
George-Miao force-pushed the refactor/runtime/timer branch from 882f1f7 to 3826863 Compare August 3, 2026 03:38
Berrysoft
Berrysoft previously approved these changes Aug 3, 2026
@Berrysoft
Berrysoft dismissed their stale review August 3, 2026 04:24

Memory leak?

Comment thread compio-runtime/src/time/future.rs
@George-Miao
George-Miao requested a review from Berrysoft August 3, 2026 21:27
@George-Miao
George-Miao enabled auto-merge (squash) August 3, 2026 22:54
Comment thread compio-runtime/src/lib.rs
Err(payload) => {
// Pending tasks must be cleared after the active unwind has
// ended so their futures can be dropped safely.
self.enter(|| self.executor.clear());

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What if we spawn a future in the first block_on calling, and await the JoinHandle in the second block_on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request package: runtime Related to compio-runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants