diff options
| author | Marco Roeland <marco.roeland@xs4all.nl> | 2006-03-11 09:55:50 +0100 |
|---|---|---|
| committer | Junio C Hamano <junkio@cox.net> | 2006-03-11 02:00:35 -0800 |
| commit | 8e7f9035b8271469747e3745aa7e095e12f74e5e (patch) | |
| tree | b57d5f313fd37b8d82689608f3c4dfceda7883cd /imap-send.c | |
| parent | be767c91724275c4534965c0d25c452b76057602 (diff) | |
| download | git-8e7f9035b8271469747e3745aa7e095e12f74e5e.tar.gz git-8e7f9035b8271469747e3745aa7e095e12f74e5e.tar.xz | |
imap-send: cleanup execl() call to use NULL sentinel instead of 0
Some versions of gcc check that calls to the exec() family have the proper
sentinel for variadic calls. This should be (char *) NULL according to the
man page. Although for all other purposes the 0 is equivalent, gcc
nevertheless does emit a warning for 0 and not for NULL. This also makes the
usage consistent throughout git.
The whitespace in function calls throughout imap-send.c has its own style,
so I left it that way.
Signed-off-by: Junio C Hamano <junkio@cox.net>
Diffstat (limited to 'imap-send.c')
| -rw-r--r-- | imap-send.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/imap-send.c b/imap-send.c index fddaac00c..203284d8f 100644 --- a/imap-send.c +++ b/imap-send.c @@ -945,7 +945,7 @@ imap_open_store( imap_server_conf_t *srvc ) _exit( 127 ); close( a[0] ); close( a[1] ); - execl( "/bin/sh", "sh", "-c", srvc->tunnel, 0 ); + execl( "/bin/sh", "sh", "-c", srvc->tunnel, NULL ); _exit( 127 ); } |