[Clam-devel] Re: Some steps to take on your own
wangjun at dsp.ac.cn
Wed Jun 11 06:36:08 PDT 2008
** Remove the remaining implementation in Contiguous, all test passing, commit
----- Original Message -----
From: "JunJun" <wangjun at dsp.ac.cn>
To: "David García Garzón" <dgarcia at iua.upf.edu>
Cc: <clam-devel at llistes.projectes.lafarga.org>
Sent: Wednesday, June 11, 2008 9:31 PM
Subject: Re: Some steps to take on your own
>> 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.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 6572 bytes
Desc: not available
More information about the clam-devel