From: Jonathan Nieder Date: Wed, 17 Nov 2010 20:15:09 GMT Subject: Re: [PATCH 4/4] builtin: use builtin.h for all builtin commands Message-ID: <20101117201509.GA27452@burratino> In-Reply-To: Ævar Arnfjörð Bjarmason wrote: > I can do that too. I just got the impression previously that we wanted > to only include things in headers that made sense according to their > name. Gettext has nothing to do with the cache, and cache.h itself > doesn't need it. A little bit of history: git was once called dircache. cache.h is practically speaking git.h, which is why it declares: - get_git_dir - ALLOC_GROW - zlib_compression_level - safe_crlf - mksnpath - git_pathdup - hashclr - validate_headref - peel_to_type - approxidate_careful - packed_git - path_match - fsync_or_die - setup_pager - checkout_fast_forward to list a random assortment. See "A birds-eye view of Git's source code" in the user manual for more on this.