-
Notifications
You must be signed in to change notification settings - Fork 108
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
Remove JobListReadLock and simplify JobQueue #6114
Conversation
@@ -89,10 +89,8 @@ def _queue_state_event_type(state: str) -> str: | |||
# pylint: disable=too-many-public-methods | |||
class JobQueue(BaseCClass): # type: ignore | |||
TYPE_NAME = "job_queue" | |||
_alloc = ResPrototype("void* job_queue_alloc()", bind=False) | |||
_alloc = ResPrototype("void* job_queue_alloc(void*)", bind=False) |
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.
Not sure if I got this CWrap thing correct here
6d5aa68
to
129e822
Compare
39067ad
to
ad65f8f
Compare
ad65f8f
to
662e95a
Compare
Codecov Report
@@ Coverage Diff @@
## main #6114 +/- ##
=======================================
Coverage 82.42% 82.42%
=======================================
Files 350 350
Lines 21457 21450 -7
Branches 839 839
=======================================
- Hits 17685 17681 -4
+ Misses 3474 3471 -3
Partials 298 298
... and 4 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Remove unused JobListReadLock
Initialize struct members
Avoid state for JobQueue where alloc returns a not fully functional object before set_driver was executed
Update pybind signatures
Pre review checklist
Ground Rules),
and changes to existing code have good test coverage.
Pre merge checklist