[vsipl++] maxval_test

Jules Bergmann jules at codesourcery.com
Mon Mar 26 18:53:12 UTC 2007


>> [1] This file should be called tests/maxval.cpp or tests/par_maxval.cpp.
>>  The _test suffix isn't necessary for files in the tests directory.
> 
> Since this test appears to explicitely test parallel functionality, may I
> suggest to put it into tests/parallel/maxal.cpp instead ?
> 
> This will be useful when / if we decide to extend our parallel testing harness
> (for example by running the tests multiple times, with differing numbers of
> processes).

Yes, that's a good idea.  Let's put this in tests/parallel.

-- 
Jules Bergmann
CodeSourcery
jules at codesourcery.com
(650) 331-3385 x705



More information about the vsipl++ mailing list