On Tue, 8 Aug 2017 13:36:24 -0700 Stefan Beller <sbeller@xxxxxxxxxx> wrote: > On Tue, Aug 8, 2017 at 12:32 PM, Jonathan Tan <jonathantanmy@xxxxxxxxxx> wrote: > > Currently, sha1_file.c and cache.h contain many functions, both related > > to and unrelated to packfiles. This makes both files very large and > > causes an unclear separation of concerns. > > > > Create a new file, pack.c, to hold all packfile-related functions > > currently in sha1_file.c, and designate pack.h to hold these > > packfile-related functions. > > There are also packed refs, so one could (like I did) think that > pack.c is for generic packing of things, maybe packfile.c > would be more clear? Good point. I'll use packfile.c and packfile.h in the next version.
