Robert de Bath <robert@xxxxxxxxxxxxxxxxx> writes: >> ... text unnecessarily opaque ... > Yes, that's quite likely. > However, I think you should continue to have a similar form to the > previous 'pack.window' and > 'pack.depth' descriptions... perhaps: > > The maximum size of memory that is consumed by each > thread in linkgit:git-pack-objects[1] for pack window memory > when no limit is given on the command line. > The value can be suffixed with "k", "m", or "g". When left > unconfigured, there will be no limit. OK, let's go with that wording. Thanks. > > Though, I suspect the existing wording is trying to avoid saying there's a 'default' and > another 'default' which would be covered by your rewording. > > So I have no real problem either way. > > Robert. > > ________________________________________ > From: Junio C Hamano [gitster@xxxxxxxxx] > Sent: 24 October 2014 18:27 > To: Robert de Bath > Cc: git@xxxxxxxxxxxxxxx > Subject: [NOTWHITELIST] Re: [PATCH] Expand on impact of threads on memory consumption. > > Robert de Bath <rdebath@xxxxxxxxxxxxxxxxx> writes: > >> diff --git a/Documentation/config.txt b/Documentation/config.txt >> index c55c22a..6384038 100644 >> --- a/Documentation/config.txt >> +++ b/Documentation/config.txt >> @@ -1828,7 +1828,10 @@ pack.depth:: >> >> pack.windowMemory:: >> The window memory size limit used by linkgit:git-pack-objects[1] >> - when no limit is given on the command line. The value can be >> + when no limit is given on the command line. This limit may be >> + allocated individually by each thread doing delta compression >> + so the total memory allocated by linkgit:git-pack-objects[1] can >> + be this multiplied by the number of threads. The value can be >> suffixed with "k", "m", or "g". Defaults to 0, meaning no >> limit. > > I sense that this change makes the resulting text unnecessarily > opaque by avoiding touching the first sentence. Wouldn't something > more concise like this sufficient? > > The default maximum size of memory that is consumed by each > thread in linkgit:git-pack-objects[1] for pack window memory. > The value can be suffixed with "k", "m", or "g". When left > unconfigured, there will be no limit. -- 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