Actually, why remove this function? Let it set `paramp->stdout_isatty = os_isatty(STDOUT_FILENO);` and let `main()` call it as before. You moved the check to later in the `main()` function after things were initialized and setup was performed. Are you sure there isn't a reason it was checked early in the function, and then the warning is displayed later?