Hi Willem
Quite a few whitespace issues in the patch, trailing whitespace,
if () {
}
else {
}
is not how we do it... but that's all easy to fix.
I still need to stare at the code some more but it looks about right.
I think it makes the algorithm overall much harder to read. The
save_pressure and save_interpolated pointers add another level of
indirection that threw me off the first two times I read through it :-/
On the flip side, the added comments are really helpful.
I'll take it but I'll clean up the whitespace before pushing it out. So be
careful with the next one in the series to make sure it's relative to what
will soon be in master :-)
/D
_______________________________________________
subsurface mailing list
[email protected]
http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface
