On Thu, Mar 8, 2012 at 12:36 PM, Dominique Quatravaux <domq@xxxxxxxxxx> wrote: > On Thu, Mar 8, 2012 at 12:20 PM, Johannes Sixt <j.sixt@xxxxxxxxxxxxx> wrote: >> Am 3/8/2012 11:42, schrieb Dominique Quatravaux: >>> + shortsha1=$(echo $sha1 | cut -c1-7) >> >>> - sha1=$(git rev-parse $shortsha1) >> >> Why do you call it "optimization" when you spend two or three subprocesses >> instead of one? > > echo is a shell internal. "git rev-parse" is two processes just as > "cut" and a pipe. My mistake, strace git rev-parse revals that this is only one process. Still, I think that saving a bunch of filesystem access beats saving one fork() (one of the two processes in my patched version is a shell, so no execve() there) but I admit I haven't benchmarked this. -- Dominique Quatravaux +41 79 609 40 72 -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html