arq5x/bedtools — comment on commit 5ff08a3d7a4bf64dcd6f879b608b1f26f249b8ea
File:  line 0
By jmarshall:

It renders fine. My first thought was horror but it's growing on me :-). If you do keep it, I'd suggest only colouring it if output is going straight to the terminal (i.e. if isatty(STDOUT_FILENO)) as if someone redirects the usage display to a file for future reference, they don't want terminal escapes in it.

More minor comments:

* Best to `#include ` explicitly now, before the next version of GCC chokes without it
* "\e" is a non-standard escape character; better as "\033"
* People will type "bedtools faq" as often as "bedtools FAQ"!
* It would be great if it also understood "bedtools --version", outputting the version number ideally in a format similar to [the GNU format](http://www.gnu.org/prep/standards/html_node/_002d_002dversion.html)

Finally, you probably regularly build it on Linux and would soon have seen this build failure anyway... is it useful to you to have these things reported?