The diffing functions take a (currently undocumented) callback option. If provided, the diff is calculated in async mode and the result is passed as the second argument to the callback function.

Unless I'm missing something, the first argument is just... never provided. It's always undefined, whether the diff is calculated successfully or not.

This seems pretty bizarre. We should just eliminate the first argument.