Re: Segmentation component to Pool

hi,
the patch for the last session, and the comments are as below:

* Jun patch. Hopefully the end for this session
** fix the bug of ContiguousSeg by moving maxPos to the last offset;
** add LoadFrom xml, with/without MaxPos, and no-within/within an array, pass 
the tests respectively.

The next target is to make an python script that changes the old segmentation 
syntax to the new one. Super!

Bests,
Jun

----- Original Message ----- 
From: "David García Garzón" <dgarcia@xxxxxxxxxxx>
To: "JunJun" <wangjun@xxxxxxxxx>
Cc: <clam-devel@xxxxxxxxxxxxxxxxxxxxxxxxxxxxx>
Sent: Wednesday, June 18, 2008 5:53 PM
Subject: Re: Segmentation component to Pool


> On Wednesday 18 June 2008 03:23:21 JunJun wrote:
>> > On Dimarts 17 Juny 2008, JunJun wrote:
>> > > Below is the statuses of each step in this patch.
>> > > storing array tests:  status- GREEN
>> > > loading maxpos:  status- GREEN
>> > > loading without maxpos:  status- GREEN
>> > > loading arrays: status- RED
>> > >
>> > > I've no idea whether the implementation of an array
>> > > "XmlStorage::Restore(segmentations, stream);" is right or not, any
>> > > idea?
>> > >
>> > > Bests,
>> > > June
>> >
>> > If the segmentations in the xml are not equal why do you expect equal
>> > segmentations on the assert sentences?
>>
>>  To make sure that it will be RED.
> 
> 
> Ah, ok, that's perfect, a RED is a required step but it should last for 
> short. ;-) But if the current implementation is also RED no need to fake the 
> test. Anyway i think that the implementation should work.
> 
> The problem is not on the implementation but again in the test. Add an assert 
> on the array size and you will get a hint on what's happening.
> 
> David.

MaxPos2XML5.patch
Description: Binary data

_______________________________________________
Clam-devel mailing list
Clam-devel-5rtuXYfMpA4ynI7IUCVSd4jY/PEAMeSRhoJSb6RVeLg@xxxxxxxxxxxxxxxx
https://llistes.projectes.lafarga.org/cgi-bin/mailman/listinfo/clam-devel