diff options
| author | Lars Schneider <larsxschneider@gmail.com> | 2016-10-16 23:20:29 (GMT) |
|---|---|---|
| committer | Junio C Hamano <gitster@pobox.com> | 2016-10-17 18:36:50 (GMT) |
| commit | 81c634e94f2fef0cec295f7554080c82bd6aeeb7 (patch) | |
| tree | ee53d4aa768f496f3f34f9945e0ff3266340d14d /builtin/remote-ext.c | |
| parent | ac2fbaa674ce3d5b5faf6a83f88cc4af1654f9cd (diff) | |
| download | git-81c634e94f2fef0cec295f7554080c82bd6aeeb7.zip git-81c634e94f2fef0cec295f7554080c82bd6aeeb7.tar.gz git-81c634e94f2fef0cec295f7554080c82bd6aeeb7.tar.bz2 | |
pkt-line: rename packet_write() to packet_write_fmt()
packet_write() should be called packet_write_fmt() because it is a
printf-like function that takes a format string as first parameter.
packet_write_fmt() should be used for text strings only. Arbitrary
binary data should use a new packet_write() function that is introduced
in a subsequent patch.
Suggested-by: Junio C Hamano <gitster@pobox.com>
Signed-off-by: Lars Schneider <larsxschneider@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'builtin/remote-ext.c')
| -rw-r--r-- | builtin/remote-ext.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/builtin/remote-ext.c b/builtin/remote-ext.c index 88eb8f9..11b48bf 100644 --- a/builtin/remote-ext.c +++ b/builtin/remote-ext.c @@ -128,9 +128,9 @@ static void send_git_request(int stdin_fd, const char *serv, const char *repo, const char *vhost) { if (!vhost) - packet_write(stdin_fd, "%s %s%c", serv, repo, 0); + packet_write_fmt(stdin_fd, "%s %s%c", serv, repo, 0); else - packet_write(stdin_fd, "%s %s%chost=%s%c", serv, repo, 0, + packet_write_fmt(stdin_fd, "%s %s%chost=%s%c", serv, repo, 0, vhost, 0); } |