[Clam-devel] Re: Some steps to take on your own

[JunJun]

> Proceed the same way with the ContiguousSegmentation

> * Proceed with the same steps and patch points than ContiguousSegmentation
>> ** Make the test red
> ** Greenify
> ** Refactor
> * You may have also bugs in there, send me a mail if so.
There is no bug, except that odd-even problem which should not be condisered as a bug.

> * After that you should have the same content on both classes, as we rely on 
> the virtual function FillArray, so we'll do a refactoring in two steps:
> ** Choose one implementation (ie. Discontinuous) and move it to the base 
> class, (no StoreOn in Discontinuous now). all test passing, commit

Here is the patch where no StoreOn in Discontiuous and all test passing.

June

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6_11_fillArrayAndDumpToXML_DiscongtinuousWithOutStoreOn.patch
Type: application/octet-stream
Size: 6636 bytes
Desc: not available
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080611/ad50276f/attachment-0003.obj>