diff options
author | Jeff King <peff@peff.net> | 2021-09-24 14:39:44 -0400 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2021-09-27 12:36:45 -0700 |
commit | 9aab952e855be1a567d4d0585afbdbde624e274f (patch) | |
tree | 03d2fd8554577fc92a1f7e53f036b48316baf17f /bisect.h | |
parent | refs-internal.h: move DO_FOR_EACH_* flags next to each other (diff) | |
download | tgif-9aab952e855be1a567d4d0585afbdbde624e274f.tar.xz |
refs-internal.h: reorganize DO_FOR_EACH_* flag documentation
The documentation for the DO_FOR_EACH_* flags is sprinkled over the
refs-internal.h file. We define the two flags in one spot, and then
describe them in more detail far away from there, in the definitions of
refs_ref_iterator_begin() and ref_iterator_advance_fn().
Let's try to organize this a bit better:
- convert the #defines to an enum. This makes it clear that they are
related, and that the enum shows the complete set of flags.
- combine all descriptions for each flag in a single spot, next to the
flag's definition
- use the enum rather than a bare int for functions which take the
flags. This helps readers realize which flags can be used.
- clarify the mention of flags for ref_iterator_advance_fn(). It does
not take flags itself, but is meant to depend on ones set up
earlier.
Signed-off-by: Jeff King <peff@peff.net>
Reviewed-by: Jonathan Tan <jonathantanmy@google.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'bisect.h')
0 files changed, 0 insertions, 0 deletions