>>> Wouldn't it work to use the TIOCGWINSZ ioctl only if isatty() reports
>>> that we're outputting to a terminal?
>>
>>
>> Good point. Updated patch attached, which imposes no limit if we're
>> not outputting to a terminal (as suggested by Mikael).
>>
>> Ok for trunk, or am I missing anything else? (Testing welcome ...)
>>
>
> needs "#include <unistd.h>" for isatty(), perhaps.

I think it is already included (via system.h).


> Otherwise looks sane at first glance.

Thanks for the feedback!

Cheers,
Janus

Reply via email to