debian/notmuch
1
0
Fork 0
mirror of https://git.notmuchmail.org/git/notmuch synced 2024-11-22 02:48:08 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

cli/insert: rename file copy function

Browse source 
The copying has nothing to do with stdin, so call it copy_fd
instead. While at it, improve documentation and reverse the
parameters, as destination is traditionally the first parameter.
This commit is contained in:
Jani Nikula 2014-09-22 11:54:55 +02:00 committed by David Bremner
parent c878081127
commit 9324e04a82
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
notmuch-insert.c
View file

@ -251,11 +251,12 @@ maildir_open_tmp_file (void *ctx, const char *dir,
return fd;
}
/* Copy the contents of standard input (fdin) into fdout.
* Returns TRUE if a non-empty file was written successfully.
* Otherwise, return FALSE. */
/*
* Copy fdin to fdout, return TRUE on success, and FALSE on errors and
* empty input.
*/
static notmuch_bool_t
copy_stdin (int fdin, int fdout)
copy_fd (int fdout, int fdin)
{
notmuch_bool_t empty = TRUE;
@ -308,7 +309,7 @@ write_message (void *ctx, int fdin, const char *dir, char **newpath)
cleanup_path = tmppath;
if (! copy_stdin (fdin, fdout))
if (! copy_fd (fdout, fdin))
goto FAIL;
if (fsync (fdout) != 0) {