Hello Edmund,
Thanks for the check. You might consider turning the patch as ready in the cf app.
Fixing the abs/hash bracketing seems clear. The wasn't sure about rewriting "mod(i, bj)" as "mod(i, j)", because there could be some convention about parameter names, but I can't think of anything the "b" could be in the case of mod.
Might be a left-over a previous version which might have used MOD(a, b), but a/b are rather used for fp numbers and i/j for integers.
-- Fabien.
