[PATCH 0/2] ref-filter: merged & no-merged touch-ups

From: Aaron Lipman <alipman88@gmail.com>
Date: Sep 18, 2020, 00:49 UTC
Message-ID: <20200918004909.32474-1-alipman88@gmail.com>
In-Reply-To: <20200916020840.84892-1-alipman88@gmail.com>

> > Even if we are not renaming
> > things that much, a locally defined preprocessor macro with shorter
> > names, defined just before the callee, would be more appropriate for
> > a case like this, with a single callee called by only one caller.

> Thanks for stating what I was planning on saying, with particular
> emphasis on keeping these #defines in the .c file rather than the .h
> file since they are not part of the public API.

Thanks for the insight into defining macros in source vs header files- I'm still in the process of learning C and figuring out how to convey purpose & encapsulation without relying on features of higher order languages I'm used to (e.g. keyword args).

Since this got moved to next, I'm not entirely clear on whether folks want me to keep tweaking this, or if the last set of comments were general advice for future contributions.

But in the interest of diligence, I'd like to offer a couple touch-ups applying Junio's suggestions.

Aaron Lipman (2):
  ref-filter: refactor do_merge_filter()
  Doc: prefer more specific file name

 Documentation/git-branch.txt                  |  2 +-
 Documentation/git-for-each-ref.txt            |  2 +-
 Documentation/git-tag.txt                     |  2 +-
 ...lters.txt => ref-reachability-filters.txt} |  0
 ref-filter.c                                  | 29 +++++++++----------
 ref-filter.h                                  |  3 --
 6 files changed, 17 insertions(+), 21 deletions(-)
 rename Documentation/{filters.txt => ref-reachability-filters.txt} (100%)

-- 
2.24.3 (Apple Git-128)