isoAdvector broken by new meaning of pos and neg
BUGFIX: In commit 7bdbab7f4eb7698f5967984186e342cac66a8e04 of the OpenFOAM Foundation code version, Henry changed the meaning of pos and neg and introduced pos0 and neg0 for the old functionality of these functions. The new meaning of pos and neg after that commit has led isoAdvector to create small spurious bubbles in the water column when snapTol was larger than zero.
The bug can be fixed by the following two code changes:
grep -rl --include *.C --include *.H --exclude-dir *Include* "pos(" $FOAM_SRC/finiteVolume/fvMatrices/solvers/isoAdvection | xargs sed -i 's/pos(/pos0(/g'
grep -rl --include *.C --include *.H --exclude-dir *Include* "neg(" $FOAM_SRC/finiteVolume/fvMatrices/solvers/isoAdvection | xargs sed -i 's/neg(/neg0(/g'
Since the bug has pretty severe consequences for isoAdvector runs, I suggest making a hotfix out of this.