With runtime logging on it smashes the stack

Also fix the pretty printer, making it output 'if' instead of 'when'.

Issue #1396

add new read_chars method, fix bug in read_char

I had made the error consistent with other errors, but forgot to
also change the tests.

Not sure how they ended up being marked as such.

They are nominal, and can safely refer to themselves.

Closes #1463

Issue #1437

(The actual comm module lives in core.)

They appear to log okay now, but I can't promise much beyond that.

@pcwalton If you feel like taking a look, I'd be grateful. Interfaces are
boxes containing a (tydesc, dict, value_of_any_type) tuple, where the leading
tydesc describes the whole tuple.

Issue #1437

Additionally reformat so that 'make check' passes.

Closes #1467

The Rust License is actually the MIT license, not BSD

Issue #1437

Closes #1462

Issue #1436

having a read_chars method is convenient and more efficient.

the old read_char method had a bug due to re-use of the 'w' width
variable as a loop counter and so was broken for wide characters,
this patch fixes that.

Tarballs for distribution will not have a git repo.