| Message ID | pull.1485.git.1677139521.gitgitgadget@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Header cleanups | expand |
On 2/23/2023 3:05 AM, Elijah Newren via GitGitGadget wrote: > This series cleans up headers a bit, trying to remove excessive dependency > on "cache.h". I created this series a while ago, but decided to clean it up > and submit it due to Emily's recent thread and suggestion that this might be > helpful to their efforts[1]. Thanks for doing this cleanup. It certainly doesn't look like an easy task. I read the patches carefully and tried to point out anything that looked fishy and not fully supported by the commit message. However, some things are probably only fishy because it was the least-complicated time to make that change. I'd be happy with this series as-is, allowing further improvements on top. I just wanted to poke a little at these bits that were not super-clear in case there was a possible improvement. Thanks, -Stolee
Maintainer note: This series cleanly merges with next, but has a minor conflict with ed/fsmonitor-inotify. The correct resolution is to take ed/fsmonitor-inotify's copy of compat/fsmonitor/fsm-settings-darwin.c, but to add a #include "git-compat-util.h" line at the top of compat/fsmonitor/fsm-settings-unix.c. This series cleans up headers a bit, trying to remove excessive dependency on "cache.h". I created this series a while ago, but decided to clean it up and submit it due to Emily's recent thread and suggestion that this might be helpful to their efforts[1]. There are many more cleanups I could do in this area, but the series is already a good size. Notes: * Big props to Dscho for gitgitgadget; being able to test on a bunch of platforms with a variety of configurations easily is a big win in general but especially for series like this one. * I used the scripts at https://github.com/newren/git/commit/db81c8d7fb554b3edf04320e218bddeb98bb6194, and some tweaks thereof, repeatedly while making this series (though they are prone to produce both false positive and false negatives, so if you use them, only use them to generate hints about which files to look at). [1] Search for "Extremely yes" in https://lore.kernel.org/git/CAJoAoZm+TkCL0Jpg_qFgKottxbtiG2QOiY0qGrz3-uQy+=waPg@mail.gmail.com/ Elijah Newren (16): treewide: ensure one of the appropriate headers is sourced first treewide: remove unnecessary git-compat-util.h includes in headers treewide: remove unnecessary cache.h includes treewide: remove unnecessary cache.h includes in source files alloc.h: move ALLOC_GROW() functions from cache.h hash.h: move some oid-related declarations from cache.h hex.h: move some hex-related declarations from cache.h pretty.h: move has_non_ascii() declaration from commit.h ident.h: move ident-related declarations out of cache.h object.h: stop depending on cache.h; make cache.h depend on object.h dir.h: refactor to no longer need to include cache.h object-store.h: move struct object_info from cache.h replace-object.h: move read_replace_refs declaration from cache.h to here treewide: replace cache.h with more direct headers, where possible Remove unnecessary includes of builtin.h diff.h: remove unnecessary include of object.h Documentation/CodingGuidelines | 8 +- add-patch.c | 2 + advice.c | 4 +- advice.h | 2 - alias.c | 4 +- alloc.c | 2 +- alloc.h | 75 +++++ apply.c | 1 + archive-tar.c | 3 +- archive.c | 3 +- attr.c | 1 + blame.h | 1 - blob.c | 2 +- builtin/blame.c | 3 +- builtin/cat-file.c | 3 + builtin/check-mailmap.c | 1 + builtin/checkout--worker.c | 1 + builtin/commit-graph.c | 1 + builtin/config.c | 3 +- builtin/credential-cache--daemon.c | 1 + builtin/fetch-pack.c | 1 + builtin/fsck.c | 1 + builtin/fsmonitor--daemon.c | 1 + builtin/grep.c | 1 + builtin/index-pack.c | 2 + builtin/log.c | 3 +- builtin/merge.c | 1 + builtin/mktree.c | 1 + builtin/mv.c | 1 + builtin/name-rev.c | 2 +- builtin/pack-objects.c | 3 +- builtin/prune.c | 1 + builtin/repack.c | 2 +- builtin/replace.c | 1 + builtin/rev-parse.c | 1 + builtin/revert.c | 3 +- builtin/rm.c | 1 + builtin/submodule--helper.c | 1 + builtin/unpack-objects.c | 1 + builtin/upload-pack.c | 1 + builtin/var.c | 1 + bulk-checkin.c | 3 +- cache-tree.c | 3 +- cache-tree.h | 1 - cache.h | 306 +------------------ cbtree.c | 1 + cbtree.h | 2 - checkout.h | 2 +- chunk-format.c | 3 +- chunk-format.h | 1 - commit-graph.h | 1 - commit-reach.c | 3 +- commit-slab-impl.h | 2 - commit.h | 1 - compat/fsmonitor/fsm-ipc-win32.c | 1 + compat/fsmonitor/fsm-settings-darwin.c | 1 + compat/mingw.c | 1 + config.c | 5 +- daemon.c | 1 + decorate.c | 3 +- delta-islands.c | 3 +- diff-merges.c | 2 + diff-no-index.c | 1 - diff.c | 1 + diff.h | 3 +- diffcore-delta.c | 2 +- diffcore-order.c | 3 +- diffcore-pickaxe.c | 4 +- diffcore-rename.c | 7 +- diffcore-rotate.c | 3 +- diffcore.h | 4 +- dir-iterator.c | 3 +- dir.c | 3 +- dir.h | 16 +- entry.h | 4 +- environment.c | 1 + ewah/bitmap.c | 3 +- ewah/ewah_bitmap.c | 2 +- fetch-pack.c | 3 +- fmt-merge-msg.c | 2 + fsck.c | 3 +- fsck.h | 1 + fsmonitor-settings.c | 3 +- git-compat-util.h | 1 + git.c | 1 + gpg-interface.c | 1 + graph.c | 3 +- hash.h | 35 ++- hashmap.c | 2 +- help.c | 4 +- hex.c | 3 +- hex.h | 84 +++++ hook.c | 5 +- http-backend.c | 3 +- ident.c | 5 +- ident.h | 67 ++++ imap-send.c | 2 +- json-writer.c | 2 +- khash.h | 1 - kwset.c | 2 +- levenshtein.c | 2 +- line-log.c | 1 + line-log.h | 1 + linear-assignment.c | 2 +- list-objects-filter-options.c | 4 +- list-objects-filter-options.h | 3 +- list-objects-filter.c | 1 + log-tree.c | 1 + mailinfo.c | 4 +- mem-pool.c | 2 +- merge-blobs.c | 2 +- merge-recursive.c | 1 - midx.c | 3 +- negotiator/noop.c | 2 +- object-file.c | 3 +- object-store.h | 128 ++++---- object.h | 22 +- oid-array.c | 3 +- oidmap.c | 2 +- oidmap.h | 1 - oidset.c | 4 +- oidtree.c | 1 + oss-fuzz/fuzz-commit-graph.c | 1 + oss-fuzz/fuzz-pack-headers.c | 1 + oss-fuzz/fuzz-pack-idx.c | 1 + pack-bitmap-write.c | 3 +- pack-bitmap.c | 3 +- pack-mtimes.h | 2 - pack-objects.c | 3 +- packfile.c | 3 +- parallel-checkout.c | 1 + pathspec.h | 5 + pkt-line.h | 1 - pretty.c | 1 + pretty.h | 4 +- prio-queue.c | 3 +- protocol-caps.c | 1 + prune-packed.c | 1 + quote.c | 2 + read-cache.c | 1 + rebase.c | 1 + ref-filter.c | 5 +- reflog-walk.c | 3 +- reflog-walk.h | 2 - refs.c | 3 +- refs/debug.c | 2 +- refs/files-backend.c | 1 + refs/packed-backend.c | 3 +- refs/ref-cache.c | 3 +- refs/refs-internal.h | 1 - refspec.c | 3 +- remote-curl.c | 3 +- remote.c | 3 +- remote.h | 1 - replace-object.h | 8 + repo-settings.c | 2 +- repository.h | 1 - rerere.c | 3 +- revision.c | 3 +- revision.h | 1 + send-pack.c | 2 +- sequencer.c | 1 + sequencer.h | 2 +- serve.c | 3 +- server-info.c | 3 +- sha1dc_git.c | 4 +- shallow.c | 3 +- shallow.h | 2 + shell.c | 2 +- sigchain.c | 3 +- sparse-index.c | 1 + split-index.c | 1 + statinfo.h | 24 ++ strbuf.c | 3 +- string-list.c | 3 +- strvec.c | 3 +- sub-process.c | 1 + sub-process.h | 1 - submodule-config.c | 1 + submodule-config.h | 1 - submodule.c | 4 +- t/helper/test-bloom.c | 2 +- t/helper/test-crontab.c | 1 - t/helper/test-ctype.c | 1 - t/helper/test-example-decorate.c | 2 +- t/helper/test-json-writer.c | 1 - t/helper/test-pcre2-config.c | 1 - t/helper/test-prio-queue.c | 1 - t/helper/test-reach.c | 2 +- t/helper/test-run-command.c | 2 - t/helper/test-sigchain.c | 1 - t/helper/test-simple-ipc.c | 3 +- t/helper/test-submodule-nested-repo-config.c | 1 + t/helper/test-wildmatch.c | 1 - thread-utils.c | 2 +- trace.h | 1 - trace2.c | 3 +- trace2/tr2_ctr.c | 2 +- trace2/tr2_sysenv.c | 2 +- trace2/tr2_tbuf.c | 2 +- trace2/tr2_tgt_event.c | 2 +- trace2/tr2_tgt_normal.c | 2 +- trace2/tr2_tgt_perf.c | 2 +- trace2/tr2_tls.c | 4 +- trace2/tr2_tmr.c | 3 +- trailer.c | 1 + transport.c | 4 +- tree-walk.c | 1 + tree-walk.h | 4 +- unix-socket.c | 3 +- unix-stream-server.c | 2 +- url.c | 4 +- urlmatch.c | 5 +- userdiff.c | 4 +- wildmatch.c | 2 +- worktree.c | 3 +- worktree.h | 1 - xdiff-interface.h | 2 +- 218 files changed, 691 insertions(+), 529 deletions(-) create mode 100644 hex.h create mode 100644 ident.h create mode 100644 statinfo.h base-commit: 06dd2baa8da4a73421b959ec026a43711b9d77f9 Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1485%2Fnewren%2Fheader-cleanups-v1 Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1485/newren/header-cleanups-v1 Pull-Request: https://github.com/gitgitgadget/git/pull/1485