[PATCH] New fstest hitting rfs bitmaps - fsrfsbitmap01

Chris Johns chrisj at rtems.org
Thu Sep 27 01:55:26 UTC 2012


Krzysztof Mięsowicz wrote:
>
> I'm sending patch with new fstest. It seems to almost completely cover
> rfs_bitmap_* symbols. This test is extended version of
> rtems_rfs_bitmap_ut.c which was written by Chris Johns.
>

Looks fine. It is nice to see this code being tested and full coverage 
be achieved.

The only reason I placed the original test code in the RFS directory was 
to contain the test in the RFS source package. If the code was moved to 
another machine to allow support on other hosts, say via FUSE, the test 
could travel with it.

We should have only one copy of the test so if the test stays in the 
testsuite tree the other file should be removed. I do not mind this 
patch being applied with the other test being removed.

> Please, take a look at it and point any mistakes.
>
> Chris, please look at copyright note and see if it is good for you, if
> not - please suggest right version :)
>

It is fine. Normally you do not remove an existing copyright notice.

Chris



More information about the devel mailing list