[PATCH v3 00/13] Reference iterators
- From
Michael Haggerty <mhagger@alum.mit.edu>
- Date
- Jun 18, 2016, 04:15 UTC
- Message-ID
- <cover.1466222921.git.mhagger@alum.mit.edu>
This is the reroll of branch mh/ref-iterators promised and discussed earlier [1]. There are no showstoppers here, but it would be preferable to merge this version rather than v2.
This version squashes in Ramsay's patch [2], addresses Eric's comments, and incorporates the whitespace fix that you made when incorporating v2.
List of changes relative to v2:
* In "refs: introduce an iterator interface":
* Make two vtables private.
* Add a whitespace fix from Junio's mh/ref-iterators.
* In "dir_iterator: new API for iterating over a directory tree":
* Add and improve some comments and docstrings.
* Fix some formatting problems.
* Use a for rather than a while loop in `dir_iterator_abort()` to
improve the clarity. * Warn on failures of `opendir()`, `readdir()`, and `closedir()`
that can't be explained as a simple race.* In "for_each_reflog(): reimplement using iterators", make the vtable private.
These changes are also available from my GitHub fork [3] as branch "ref-iterators".
Michael
[1] http://article.gmane.org/gmane.comp.version-control.git/296883 [2] http://thread.gmane.org/gmane.comp.version-control.git/296801 [3] https://github.com/mhagger/git
Michael Haggerty (13):
refs: remove unnecessary "extern" keywords
do_for_each_ref(): move docstring to the header file
refs: use name "prefix" consistently
delete_refs(): add a flags argument
remote rm: handle symbolic refs correctly
get_ref_cache(): only create an instance if there is a submodule
entry_resolves_to_object(): rename function from
ref_resolves_to_object()
ref_resolves_to_object(): new function
refs: introduce an iterator interface
do_for_each_ref(): reimplement using reference iteration
for_each_reflog(): don't abort for bad references
dir_iterator: new API for iterating over a directory tree
for_each_reflog(): reimplement using iteratorsMakefile | 2 + builtin/fetch.c | 2 +- builtin/remote.c | 8 +- dir-iterator.c | 202 +++++++++++++++++ dir-iterator.h | 87 +++++++ iterator.h | 81 +++++++ refs.c | 20 ++ refs.h | 139 +++++++----- refs/files-backend.c | 629 +++++++++++++++++++++++++++++++-------------------- refs/iterator.c | 384 +++++++++++++++++++++++++++++++ refs/refs-internal.h | 226 +++++++++++++++++- 11 files changed, 1458 insertions(+), 322 deletions(-) create mode 100644 dir-iterator.c create mode 100644 dir-iterator.h create mode 100644 iterator.h create mode 100644 refs/iterator.c
-- 2.8.1