More dive-site-management goodies.

Linus Torvalds torvalds at linux-foundation.org
Mon Jun 1 17:12:57 PDT 2015


On Mon, Jun 1, 2015 at 5:03 PM, Dirk Hohndel <dirk at hohndel.org> wrote:
>
> And the fix is pushed. Can you check that it's correct, Linus? Works here
> with your data file.

Works for me, and looks correct. I actually thought that "-1" was
wrong to begin with, but then decided that the whole
"beginInsertRows()" must be the cause ("Maybe it increments the
number"), and that the "-1" was trying to correct for some Qt issue.

Because yes, the end of std::sort() is supposed to be the end of the
array, not the pointer to the last entry.

                   Linus


More information about the subsurface mailing list