Hi Junio, On 2015-10-07 19:49, Junio C Hamano wrote: > Johannes Schindelin <johannes.schindelin@xxxxxx> writes: > >> On Windows, files that are in use cannot be removed or renamed. That >> means that we have to release pack files when we are about to, say, >> repack them. Let's introduce a convenient function to close them >> pack files. >> >> While at it, we consolidate the close windows/close fd/close index >> stanza in `free_pack_by_name()` into the `close_pack()` function that >> is used by the new `close_all_packs()` function to avoid repeated code. >> >> Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx> >> --- > > This is the only one that was updated among the four and the change > looks sensible. I'd reword the end of the first paragraph of the > proposed log message, though, to replace "close them pack files" > with "close all the pack files and their idx files". > > Thanks. This time I was more clever and checked `pu` before sending off a fixed patch, seeing that you patched this already. Thanks, Dscho -- 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