Well,

As far as I get it, the equals method compares the Comparator itself and not the 
RowRecords. I simply followed a code example from
the HSSF project (DirectoryProperty) and it seems quite logical.

What do u think ?
Tal

----- Original Message -----
From: "Avik Sengupta" <[EMAIL PROTECTED]>
To: "POI Developers List" <[EMAIL PROTECTED]>
Sent: Wednesday, May 15, 2002 11:43 PM
Subject: Re: RowRecordAggregate patch


> Thanks.
>
> A couple of questions. Why is there an implementation of equals in the
> RowRecordComparator class, that does exactly what Object.equals does?
>
> Since RowRecordComparator compares RowRecords in order of their row numbers,
> shouldnt there be an equals implementation of RowRecords,  that equates them on
> the same basis (ie, row nums)? else there is the possibility of the map
> misbehaving, if the Comparator is not "consistent with equals"
>
> -
> Avik
>
>
> Quoting Tal Moshaiov <[EMAIL PROTECTED]>:
>
> > Hello Group !
> >
> > Attached is what I believe to be a bug fix for the RowRecordAggregate class.
> >
> > The bug resides in the fact that when asked to "getRow(rowNum)"  the
> > comparison in the Map is made by Objects and by numbers.
> >
> > Hope this is ok.
> >
> > Regards
> > Tal
> >
> > ----------------------------------
> > Tal Moshaiov
> > Applications Development
> > VistaPortal Software, Inc.
> > [EMAIL PROTECTED]
> > Phone: 972-3-6126188
> > Fax:      972-3-6126189
> > ICQ:      109104838
> > www.vistaportal.com
> >
> >
>
>
>
> --
> To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to