git/daemon.c, branch v2.1.2

Merge branch 'cc/replace-edit'

2014-07-16T18:25:47Z

Teach "git replace" an "--edit" mode. * cc/replace-edit: replace: use argv_array in export_object avoid double close of descriptors handed to run_command replace: replace spaces with tabs in indentation

avoid double close of descriptors handed to run_command

2014-06-25T22:27:24Z

When a file descriptor is given to run_command via the "in", "out", or "err" parameters, run_command takes ownership. The descriptor will be closed in the parent process whether the process is spawned successfully or not, and closing it again is wrong. In practice this has not caused problems, because we usually close() right after start_command returns, meaning no other code has opened a descriptor in the meantime. So we just get EBADF and ignore it (rather than accidentally closing somebody else's descriptor!). Signed-off-by: Jeff King Signed-off-by: Junio C Hamano

daemon: use skip_prefix to avoid magic numbers

2014-06-20T17:45:18Z

Like earlier cases, we can use skip_prefix to avoid magic numbers that must match the length of starts_with prefixes. However, the numbers are a little more complicated here, as we keep parsing past the prefix. We can solve it by keeping a running pointer as we parse; its final value is the location we want. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano

use skip_prefix to avoid magic numbers

2014-06-20T17:44:45Z

It's a common idiom to match a prefix and then skip past it with a magic number, like: if (starts_with(foo, "bar")) foo += 3; This is easy to get wrong, since you have to count the prefix string yourself, and there's no compiler check if the string changes. We can use skip_prefix to avoid the magic numbers here. Note that some of these conversions could be much shorter. For example: if (starts_with(arg, "--foo=")) { bar = arg + 6; continue; } could become: if (skip_prefix(arg, "--foo=", &bar)) continue; However, I have left it as: if (skip_prefix(arg, "--foo=", &v)) { bar = v; continue; } to visually match nearby cases which need to actually process the string. Like: if (skip_prefix(arg, "--foo=", &v)) { bar = atoi(v); continue; } Signed-off-by: Jeff King Signed-off-by: Junio C Hamano

daemon: mark some strings as const

2014-06-18T21:56:24Z

None of these strings is modified; marking them as const will help later refactoring. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano

Merge branch 'jk/daemon-tolower'

2014-06-16T17:07:15Z

* jk/daemon-tolower: daemon/config: factor out duplicate xstrdup_tolower

daemon/config: factor out duplicate xstrdup_tolower

2014-05-23T19:39:44Z

We have two implementations of the same function; let's drop that to one. We take the name from daemon.c, but the implementation (which is just slightly more efficient) from the config code. Signed-off-by: Jeff King Signed-off-by: Junio C Hamano

daemon: move daemonize() to libgit.a

2014-02-10T18:46:35Z

Signed-off-by: Nguyễn Thái Ngọc Duy Signed-off-by: Junio C Hamano

Merge branch 'nd/daemon-informative-errors-typofix'

2014-01-10T18:32:59Z

* nd/daemon-informative-errors-typofix: daemon: be strict at parsing parameters --[no-]informative-errors

daemon: be strict at parsing parameters --[no-]informative-errors

2013-12-20T22:05:07Z

Use strcmp() instead of starts_with()/!prefixcmp() to stop accepting --informative-errors-just-a-little Signed-off-by: Nguyễn Thái Ngọc Duy Signed-off-by: Junio C Hamano