Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> writes: > Signed-off-by: Nguyễn Thái Ngọc Duy <pclouds@xxxxxxxxx> > --- > cache.h | 3 +++ > entry.c | 39 ++++++++++++++++++++++++++------------- > 2 files changed, 29 insertions(+), 13 deletions(-) It was the goal of the original streaming output topic to helping more callers stream the data out directly from the object store in order to reduce memory pressure, and this series is very much in line with its spirit. The static version of streaming_write_entry() in entry.c was very specific to writing out an index entry out to the working tree, and it made perfect sense to have the function in that file, but its interface was limited to the original context the function was used in. The whole point of your refactoring in this patch is to make it available for callers outside that original context; e.g. archive that finds blob SHA-1 from a tree and writes the blob out to its standard output. They should not have to work with an API that takes a cache-entry and writes to a working tree file. And your result is much more generic. So I think the external declaration and the definition should move to a more generic place, namely streaming.[ch]. It does not belong to entry.c anymore. Thanks for working on this. -- 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