Merge branch 'jk/pager-in-use'
authorJunio C Hamano <gitster@pobox.com>
Tue, 28 Mar 2017 21:05:58 +0000 (14:05 -0700)
committerJunio C Hamano <gitster@pobox.com>
Tue, 28 Mar 2017 21:05:59 +0000 (14:05 -0700)
Code clean-up.

* jk/pager-in-use:
pager_in_use: use git_env_bool()

1  2 
pager.c
diff --combined pager.c
index 73ca8bc3b172e62dd4ee81a5490a9c3f067b3626,ec85fdf4c6a79433972ba164e695cae035109123..c113d898a4ab184cf5ad5ea04dc66365fdf8a8f8
+++ b/pager.c
@@@ -43,6 -43,37 +43,6 @@@ static int core_pager_config(const cha
        return 0;
  }
  
 -static void read_early_config(config_fn_t cb, void *data)
 -{
 -      git_config_with_options(cb, data, NULL, 1);
 -
 -      /*
 -       * Note that this is a really dirty hack that does the wrong thing in
 -       * many cases. The crux of the problem is that we cannot run
 -       * setup_git_directory() early on in git's setup, so we have no idea if
 -       * we are in a repository or not, and therefore are not sure whether
 -       * and how to read repository-local config.
 -       *
 -       * So if we _aren't_ in a repository (or we are but we would reject its
 -       * core.repositoryformatversion), we'll read whatever is in .git/config
 -       * blindly. Similarly, if we _are_ in a repository, but not at the
 -       * root, we'll fail to find .git/config (because it's really
 -       * ../.git/config, etc). See t7006 for a complete set of failures.
 -       *
 -       * However, we have historically provided this hack because it does
 -       * work some of the time (namely when you are at the top-level of a
 -       * valid repository), and would rarely make things worse (i.e., you do
 -       * not generally have a .git/config file sitting around).
 -       */
 -      if (!startup_info->have_repository) {
 -              struct git_config_source repo_config;
 -
 -              memset(&repo_config, 0, sizeof(repo_config));
 -              repo_config.file = ".git/config";
 -              git_config_with_options(cb, data, &repo_config, 1);
 -      }
 -}
 -
  const char *git_pager(int stdout_is_tty)
  {
        const char *pager;
@@@ -135,9 -166,7 +135,7 @@@ void setup_pager(void
  
  int pager_in_use(void)
  {
-       const char *env;
-       env = getenv("GIT_PAGER_IN_USE");
-       return env ? git_config_bool("GIT_PAGER_IN_USE", env) : 0;
+       return git_env_bool("GIT_PAGER_IN_USE", 0);
  }
  
  /*