Brian Schott

What's going on with the auto-tester? It looks like this has both passed and failed on some of the FreeBSD 32-bit machines.

Looks like `size_t.max`. Could be either a bug in the lexer or a bug in the argument check.

This feature was already implemented long ago. The problem is that from the AST you can't tell that `d.popFront` is a function call.

The format for that file is one path per line. It's not comma-separated.

Try this: ``` /Library/D/dmd/src/phobos /Library/D/dmd/src/druntime/import /Users/{user}/.dub/packages/rainbow-master/src ```

I think one of them is that the documentation for the config file path is wrong in the README...

Given that this should be very rare (Who defines a non-const overload of the `*` operator? And if it's not const, why ignore the result?) I'm not going to prioritize...

This is not going to be easy to implement at all.

> But --keep_line_breaks adding an additional newline is definitely a bug, so I fixed only that part. The documentation states that the option means "Keep existing line breaks if these...

I've been putting off looking at this for two reasons. 1) The first is that nobody seems to have actually read my original comment on this pull request. The `keep_line_breaks`...