vlcrs-core: object: add Object::new()
This allows creating objects from existing objects (in a libvlc tree) or dummy objects (parent = None) for tests, and provide the Drop implementation to delete the objects when they go out of scope.
The implementation is specifically made to be too restrictive for now, which makes it unpractical to other use case than creating tests, since we cannot guarantee the correct usage of the object tree as it is otherwise.
Merge request reports
Activity
changed milestone to %4.0
added Component::Bindings: Rust label
- Resolved by Alexandre Janniaux
Depends on !6673 (merged)
added MRStatus::Reviewable label
- Resolved by Rémi Denis-Courmont
IMO, or rather AFAICT, we need two types for VLC objects, one for the common data and one for whole subtyped objects, e.g.
ObjectCommon
andObject<T = (): ?Sized>
. The later can implementAsMut<ObjectCommon>
andDerefMut<Target = T>
for convenience.Edited by Rémi Denis-Courmont
added MRStatus::InReview label and removed MRStatus::Reviewable label
added MRStatus::NotCompliant label and removed MRStatus::InReview label
added 322 commits
-
1f31c4a6...9dc00813 - 321 commits from branch
videolan:master
- a66b0c08 - vlcrs-core: object: add Object::new()
-
1f31c4a6...9dc00813 - 321 commits from branch
added MRStatus::Acceptable label and removed MRStatus::NotCompliant label
added MRStatus::Accepted label and removed MRStatus::Acceptable label
MR Acceptance result
This MergeRequest has been Accepted! Congratulations.MR acceptance checks details:
-
MR should be considered mergeable by Gitlab -
Last pipeline should be successful -
MergeRequest should have at least one external review and/or vote -
All threads should be resolved, and score >= 0 -
MergeRequest should have no activity (threads/votes) for (24h/24h)
-
added 26 commits
-
a66b0c08...03233bec - 25 commits from branch
videolan:master
- 76dee610 - vlcrs-core: object: add Object::new()
-
a66b0c08...03233bec - 25 commits from branch
enabled an automatic merge when all merge checks for 76dee610 pass