Re: [PATCH 4/4] builtin: use builtin.h for all builtin commands

From: Jonathan Nieder <jrnieder@gmail.com>
Date: Nov 17, 2010, 20:15 UTC
Message-ID: <20101117201509.GA27452@burratino>
In-Reply-To: <AANLkTikw0+FGJmj7C0B9J_3ZVXrbqCw+UzyP3sqfyVdm@mail.gmail.com>

Æ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.