[PATCH 0/9] fix some warnings
Dirk Hohndel
dirk at hohndel.org
Mon Oct 27 07:41:06 PDT 2014
Hi Lubomir...
good to have you back with patches!
On Mon, Oct 27, 2014 at 11:41:53AM +0200, Lubomir I. Ivanov wrote:
> >> please ignore anything in particular if you see fit.
> >> i'm not really advocating any of these.
I liked all of them except for one...
> >> [PATCH 5/9] divelogexportdialog.cpp: attempt to fix a 'unsigned /
The original code clearly has issues, but so does your patch. Something is
fishy if the offset can be negative but we compare it against a time
that's unsigned. I need to stare at the code and look at the values a bit
longer - or maybe you could do that (as I won't have time until late
tonight, I think).
Everything else I will push in a moment once my test builds finish.
/D
More information about the subsurface
mailing list