cmake -DMARBLE_INCLUDE_DIR broken?

Dirk Hohndel dirk at hohndel.org
Thu Sep 24 07:10:55 PDT 2015


On Wed, Sep 23, 2015 at 09:17:24PM -1000, Gaetan Bisson wrote:
> [2015-09-23 20:10:35 -1000] Gaetan Bisson:
> > So I do not change the dive site at all (just fiddle with the notes),
> > and when I click "Apply Changes", in MainTab::updateDiveSite(), the
> > pickedUuid variable is 16777215 == 2^24-1. I'm not sure how that
> > happens, but obviously it should be zero in order to be caught by the
> > test you added.
> 
> Ha! The currUuid variable was never initialized...

Bah. I should have caught that in code review :-(
 
> Patch attached.

Thanks. Applied.

/D


More information about the subsurface mailing list