Bug 9114

Summary: [PATCH] fix RowRecordAggregate.getRow bug
Product: POI Reporter: Tal Moshaiov <talm>
Component: HSSFAssignee: POI Developers List <dev>
Status: RESOLVED FIXED    
Severity: normal CC: glens
Priority: P3    
Version: 2.0-dev   
Target Milestone: ---   
Hardware: All   
OS: All   
Attachments: cvs diff file
ooops forgot to add @author in the last one

Description Tal Moshaiov 2002-05-15 15:06:43 UTC
I believe this is a bug in RowRecordAggregate.
The bug resides in the fact that when asked to "getRow(rowNum)"  the comparison
in the HashMap is made by Objects comparison instead of row numbers comparison.

Greetz
Tal
Comment 1 Tal Moshaiov 2002-05-15 15:07:52 UTC
Created attachment 1873 [details]
cvs diff file
Comment 2 Tal Moshaiov 2002-05-15 15:09:27 UTC
Created attachment 1874 [details]
ooops forgot to add @author in the last one
Comment 3 Andy Oliver 2002-05-20 21:05:03 UTC
Glen, as I understand it you had concerns with this patch?  Have they been resolved?

Tal, is this patch against 1.5 or against the head?  Are you intending for it to
be applied and appear in 2.0 or a possible later 1.5 bugfix release? 

Once I get confirmation from you gentlemen on these issues I'll apply and commit.

Thanks,

Andy
Comment 4 Andy Oliver 2002-05-20 21:06:43 UTC
Glen, as I understand it you had concerns with this patch?  Have they been resolved?

Tal, is this patch against 1.5 or against the head?  Are you intending for it to
be applied and appear in 2.0 or a possible later 1.5 bugfix release? 

Once I get confirmation from you gentlemen on these issues I'll apply and commit.

Thanks,

Andy
Comment 5 Glen Stampoultzis 2002-05-21 00:35:17 UTC
Did I?  I don't recall.  Been a bit slow keeping up with this stuff lately.  
This will change soon.
Comment 6 Avik Sengupta 2002-05-21 04:42:56 UTC
I had problems, and they havent been resolved. I'll get to them later today.
Comment 7 Andy Oliver 2002-05-21 13:00:43 UTC
OK, I'll make sure Tal looks at this and tells me what version he intended it
for and then I'll apply.
Comment 8 Avik Sengupta 2002-05-21 19:02:10 UTC
changed RowRecordAggregate to use treemap rather than hashmap. Did not use the 
comparator.