Hi,
Sean has left the following comment at Create test for thread creation on
Windows http://www.google-melange.com/gci/task/view/google/gci2012/8086204:
unnecessary tests
I think there is some misunderstanding on your part of what bu_parallel()
does. All of the additional tests you added are rather unnecessary
(unhelpful even) and I think just reinforce the misunderstanding for what
the two parameters you've defined actually do. Calling bu_parallel() just
one time runs a function X times in parallel. So in actuality, you only
needed to have your test program call bu_parallel once. It's okay to re-run
it in a loop but from a testing perspective, you have to think about what
it's testing and hoping to catch.
On that point, your test should have included some means to verify that
bu_parallel() actually ran a function X times but I let it pass since you
got enough scaffolding in place for us to make that edit. You run it and
print a message, but don't actually have the program test anything (beyond
it compiles and didn't crash). A human becomes the tester, needing to read
the print statements and discern if anything is wrong. Still, easy to fix.
Thanks for your efforts. Again, it's a good stub to get us started (and why
it's worth closing). You've put good work in spite of the
misunderstanding. ;)
Greetings,
The Google Open Source Programs Team
---
You are receiving this message because you are subscribed to Create test
for thread creation on Windows.
To stop receiving these messages, go to:
http://www.google-melange.com/gci/task/view/google/gci2012/8086204.
------------------------------------------------------------------------------
Master Java SE, Java EE, Eclipse, Spring, Hibernate, JavaScript, jQuery
and much more. Keep your Java skills current with LearnJavaNow -
200+ hours of step-by-step video tutorials by Java experts.
SALE $49.99 this month only -- learn more at:
http://p.sf.net/sfu/learnmore_122612
_______________________________________________
BRL-CAD Tracker mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-tracker
