Joachim B Haga <cjhaga@xxxxxxxxxx> writes: > This one I'm not so sure about, it's for completeness. But I don't actually use > git and haven't tested beyond the git add / git commit stage. Still... > > Signed-off-by: Joachim B Haga (cjhaga@xxxxxxxxxx) You made a good judgement to notice that these three are different. * sha1write_compressed() in csum-file.c is for producing packs and most of the things we compress there are deltas and less compressible, so even when core.compression is set to high we might be better off using faster compression. * diff's deflate_it() is about producing binary diffs (later encoded in base85) for textual transfer. Again it is almost always used to compress deltas so the same comment as above apply to this. * http-push uses it to send compressed whole object, and this is only used over the network, so it is plausible that the user would want to use different compression level than the usual core.compression. It is fine by me to use the same core.compression to these three. If somebody comes up with a workload that benefits from having different settings for them, we can add separate variables, falling back on the default core.compression if there isn't one, as needed. Thanks for the patches. - : 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