Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
External Execution Interface #4616
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
External Execution Interface #4616
Changes from all commits
448184f
442c928
e0f9d13
adb9ad9
882e630
da9e0b2
94efc44
9695592
4554407
40d1e84
74835cf
f73a471
b743020
dd7abe0
be0855e
919ae67
0e4b73d
94b4c3f
446a9f3
f39e7f8
b15ba57
4a86d10
bd22e7d
c5b97f8
d2d209c
3264294
9603086
a06aa81
8c7c05b
8948a57
2e1bf82
74b8cb9
9ccc97d
4cab2f5
7c4b651
ad32894
2039771
a44b999
da26362
e5731f4
38feb45
fd4a623
630aa78
f387952
cfefdf8
2924b64
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can user quic callback event be invoked on this user thread here?
(I am thinking about if it is possible to write a single thread application, and potential deadlocks.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the app I added in this PR is single threaded. It works and I have the logs to prove it.
Check warning on line 145 in src/core/library.c
src/core/library.c#L143-L145
Check warning on line 723 in src/core/library.c
src/core/library.c#L721-L723
Check warning on line 752 in src/core/library.c
src/core/library.c#L750-L752
Check warning on line 2472 in src/core/library.c
src/core/library.c#L2471-L2472
Check warning on line 2478 in src/core/library.c
src/core/library.c#L2478
Check warning on line 2483 in src/core/library.c
src/core/library.c#L2483
Check warning on line 2488 in src/core/library.c
src/core/library.c#L2488
Check warning on line 2490 in src/core/library.c
src/core/library.c#L2490
Check warning on line 2495 in src/core/library.c
src/core/library.c#L2494-L2495
Check warning on line 2498 in src/core/library.c
src/core/library.c#L2497-L2498
Check warning on line 2501 in src/core/library.c
src/core/library.c#L2501
Check warning on line 2504 in src/core/library.c
src/core/library.c#L2504
Check warning on line 2510 in src/core/library.c
src/core/library.c#L2509-L2510
Check warning on line 2520 in src/core/library.c
src/core/library.c#L2519-L2520
Check warning on line 2529 in src/core/library.c
src/core/library.c#L2528-L2529
Check warning on line 2531 in src/core/library.c
src/core/library.c#L2531
Check warning on line 2534 in src/core/library.c
src/core/library.c#L2534
Check warning on line 2543 in src/core/library.c
src/core/library.c#L2542-L2543
Check warning on line 2549 in src/core/library.c
src/core/library.c#L2549
Check warning on line 2551 in src/core/library.c
src/core/library.c#L2551
Check warning on line 2556 in src/core/library.c
src/core/library.c#L2555-L2556