Hi, Luca,
Thanks for the bug fix. Apparently the same error has been replicated
once more in the same source file;-) Thanks for digging into this.
It is very helpful to us. The corrections will be in the next release
which is expected in a week or two.
John
On 11/1/2009 5:16 AM, Luca Deri wrote:
> Dear all,
> while playing with ibis::bundle I think I have found a bug in the
> reorder() method. In particular the code (bundle.ccp line 1416 and
> followings)
>
> if (direction < 0) { // reverse the order
>
>
> for (uint32_t j = 0; j < cols.size(); ++ j)
> for (uint32_t i = 0; i < ngroups/2; ++ i)
> cols[i]->swap(i, ngroups-1-i);
> should be
> cols[j]->swap(i, ngroups-1-i);
>
> With his fix reordering with two or more columns works well, whereas
> with the original code I get a core dump.
>
> Regards Luca
>
> ---
> If you can not measure it, you can not improve it - Lord Kelvin
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> FastBit-users mailing list
> [email protected]
> https://hpcrdm.lbl.gov/cgi-bin/mailman/listinfo/fastbit-users
_______________________________________________
FastBit-users mailing list
[email protected]
https://hpcrdm.lbl.gov/cgi-bin/mailman/listinfo/fastbit-users
