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

[JunJun]

** 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.
> 
> June
> 
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 6_11_fillArrayAndDumpToXML_NoStoreOnInDerivedClasses.patch
Type: application/octet-stream
Size: 6572 bytes
Desc: not available
URL: <http://lists.clam-project.org/pipermail/clam-devel-clam-project.org/attachments/20080611/00ba24d7/attachment-0004.obj>