sed-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

suggestion: more tests + question


From: Assaf Gordon
Subject: suggestion: more tests + question
Date: Fri, 27 May 2016 00:53:39 -0400

Hello,

Attached are patches to add few more tests (currently untested code, based on 
'make coverage').
Each new test in a separate commit just to ease review.

While reading the code, I noticed that 'util.c:register_open_file()' always 
sets "p->temp = false",
regardless of the value of the 'temp' parameter.
Since that is the only place that marks the files as temporary, could it mean 
that no files are actually marked as temporary? leading to 'utils.c:panic()' 
never to unlink any temp file?



- assaf


Attachment: sed-tests-2016-05-26.patch
Description: Binary data


reply via email to

[Prev in Thread] Current Thread [Next in Thread]