I'm aware of the pending patch to improve color support on Windows. I'll check that one out as well, but it appears to be orthogonal to this one.
Actually I think it would be better to rebase that patch on top of this, as the Windows function enable_vt_mode() incorporates the logic of both isatty() and terminal_supports_color() by enabling CMDs support of VT100 escape codes.
