Re: [PATCH xtest] Allow space in read buffer for CRLF and NUL

2012-12-13 Thread Peter Harris
On 2012-12-06 18:38, Aaron Plattner wrote: Reviewed-by: Aaron Plattner aplatt...@nvidia.com Thanks. Pushed. Peter Harris -- Open Text Connectivity Solutions Group Peter Harrishttp://connectivity.opentext.com/ Research and DevelopmentPhone: +1 905 762

Re: [PATCH xtest] Allow space in read buffer for CRLF and NUL

2012-12-06 Thread Aaron Plattner
I'll take your word for it that tcc does what the message says. A comment above linebuf mentioning why it's 515 bytes might be nice, but in any case, Reviewed-by: Aaron Plattner aplatt...@nvidia.com On 12/04/2012 03:38 PM, Peter Harris wrote: tcc will limit output lines to 512 characters

[PATCH xtest] Allow space in read buffer for CRLF and NUL

2012-12-04 Thread Peter Harris
tcc will limit output lines to 512 characters not including CRLF. In addition, there needs to be space in the buffer for the trailing NUL that fgets will append. Without this change, xts-report will exit prematurely if it sees a maximum length line. Signed-off-by: Peter Harris