Running s_all on a clean checkout is currently giving:

s_all run of: "./s_docs" resulted in:
    /home/ubuntu/dev/pm-3094/wt-clean/src/include/wiredtiger.in:4018: error: WT_COLLATOR::compare has @param documentation sections but no arguments (warning treated as error, aborting now)

Naturally we should fix this, but we should also have s_all return a non-zero value in this case. After running the above, echo $? yields 0, which is clearly wrong. The scope for this ticket is to fix both of these problems.