All the changes in this GPR relate to reducing line-lengths and fixing whitespace.

@dra27 I think we should drop this PR and try again later (maybe piecewise?)

I don't think it needs breaking up - just rebasing (and updating).

But won't it be too hard to rebase?

No - I've already done it! Just testing it... it's revealed a couple of little changes to ocamltest, so once the whole testsuite is passing again, I'll split the new bits into separate pull requests and push the new version of this one just with whitespace/line-length altering changes.

OK, this single commit contains fixes which are solely the correction of whitespace (with occasional knock-on consequences for the testsuite reference files) and overly long lines. Overly long lines are fixed using (hopefully) non-controversial string-splitting, etc.

This diff is much easier to look at using a whitespace-eliminating diff tool such as patdiff

A clearer view of this diff may be found at http://people.ds.cam.ac.uk/dra27/gpr1288-patdiff.html

Rebased, partly to kick AppVeyor, and partly because this branch should now represent a fully check-typo passing trunk

OK - assuming the revised version with GPR#1825 whitespace fixes passes, can this be merged? The patdiff-generated diff above is still correct (rebasing didn't have any conflicts).

Thanks, @damiendoligez! Here goes, and apologies to everyone whose patches need updating...