Misc #13541
closed
[PATCH] test/thread: relax checks for internal implementation
Added by normalperson (Eric Wong) about 7 years ago.
Updated about 7 years ago.
Description
Queue, SizedQueue, and ConditionVariable internal implementation
should not be tied to using arrays. Implementation details can
change and I am working on a change to remove the dependency on
arrays (as others may attempt, too) by using ccan/list for the
waiter list.
- test/thread/test_cv.rb (test_dump): remove check for Array
- test/thread/test_queue.rb (test_dump): ditto
I have no idea if my work-in-progress change to use ccan/list
will be acceptable; regardless, I do not believe we should be
testing for this implementation detail. Will commit in a few
days if no objections.
Files
It seems no problem.
I think you can write that this change is related to "error message" in commit log (dependencies caused by error message). Because I can't imagine these dependencies with your report (this ticket) before I check your patch.
Thanks,
Koichi
- Status changed from Open to Closed
Applied in changeset trunk|r58644.
test/thread: relax internal implementation check in error message
Queue, SizedQueue, and ConditionVariable internal implementation
should not be tied to using arrays. Implementation details can
change and I am working on a change to remove the dependency on
arrays (as others may attempt, too) by using ccan/list for the
waiter list.
- test/thread/test_cv.rb (test_dump): remove check for Array
- test/thread/test_queue.rb (test_dump): ditto
[ruby-core:81001] [Misc #13541]
Also available in: Atom
PDF
Like0
Like0Like0