[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
sed-tests-2016-05-26.patch
Description: Binary data
- suggestion: more tests + question,
Assaf Gordon <=