Use stdout instead of stdin for tty color checking

[Deleted user]

Wouldn't it be better to check if stdout was a tty instead of using stdin when testing to see if output should be colorized?

[grassator]

I'm not on expert in C and terminal behaviour, so I just looked around for samples of code and all that I can find, points to tests for stdin

[Deleted user]

If you run the test executable with the output redirected to a file or piped to another process, then stdout will not be a tty, but stdin will be. Generally, this is when you do not want to see the escape codes.

For example, ls from coreutils uses stdout: https://github.com/coreutils/coreutils/blob/master/src/ls.c#L2027

[grassator]

@dales-at-avid Makes, sense, made the change