This is the tenth version of a patch series that implements the GSoC microproject of converting a recursive call to readdir() to use dir_iterator. v1: https://public-inbox.org/git/CAGZ79kZwT-9mHTiOJ5CEjk2wDFkn6+NcogjX0=vjhsAh16ANYg@xxxxxxxxxxxxxx/T/#t v2: https://public-inbox.org/git/CACsJy8Dxh-QPBBLfaFWPAWUsbA9GVXA7x+mXLjEvYKhk1zOpig@xxxxxxxxxxxxxx/T/#t v3: https://public-inbox.org/git/CAGZ79kYtpmURSQWPumobA=e3JBFjKhWCdv_LPhKCd71ZRwMovA@xxxxxxxxxxxxxx/T/#t v4: https://public-inbox.org/git/1490747533-89143-1-git-send-email-bnmvco@xxxxxxxxx/T/#e437a63e0c22c00c69b5d92977c9b438ed2b9fd3a v5: https://public-inbox.org/git/1490844730-47634-1-git-send-email-bnmvco@xxxxxxxxx/T/#m2323f15e45de699f2e09364f40a62e17047cf453 v6: https://public-inbox.org/git/1491107726-21504-1-git-send-email-bnmvco@xxxxxxxxx/T/#t v7: https://public-inbox.org/git/1491163388-41255-1-git-send-email-bnmvco@xxxxxxxxx/T/#t v8: https://public-inbox.org/git/a60b2ed6-2b99-b134-05af-7c8492a6949c@xxxxxxxxxxxx/T/#t v9: https://public-inbox.org/git/CAGZ79kaBRS0SFAvrV4mN7-mVk+8QmPKPJMD55zPQ+A14ZzYFYA@xxxxxxxxxxxxxx/T/#me8988b7dd4adbc4ea24946ccb24fc1cf7baf44e3 Travis CI build: https://travis-ci.org/theiostream/git/builds/223542902 I do not know if "queuing" meant I did not have to change this series any further (specially after Stefan's "ok"), but anyway, this series applies Junio's corrections back from v9, that were mostly regarding commit messages or style. I hope I got them right. The only point I was in doubt was about Michael's signoff. Actually, he gave it not regarding the snippet for the new dir_iterator_advance() logic, but to a small piece of actual code he wrote on the new dir iterator test helper[1]. Thus I don't know whether this would require his signoff to come before or after mine. Regardless, proper credit has been given in the commit message, as suggested. In the end, I kept his before mine, but I suppose that can be adjusted by Junio if necessary. I also didn't get whether I myself should have renamed t0065 to t0066 given the other queued patch. I kept it as t0065 since I figured it would be weird for this patch as a unit to "skip" a number. Once again, thanks for all the time invested in the reviews for this patch. [1]: https://public-inbox.org/git/1491163388-41255-1-git-send-email-bnmvco@xxxxxxxxx/T/#m187b9e681e3369862ccc6083bbf6596cd2e19cd4 Daniel Ferreira (5): dir_iterator: add tests for dir_iterator API remove_subtree(): test removing nested directories dir_iterator: refactor dir_iterator_advance dir_iterator: rewrite state machine model remove_subtree(): reimplement using iterators Makefile | 1 + dir-iterator.c | 252 +++++++++++++++++++++++++++++----------- dir-iterator.h | 35 ++++-- entry.c | 42 +++---- refs/files-backend.c | 51 +++++--- t/helper/.gitignore | 1 + t/helper/test-dir-iterator.c | 53 +++++++++ t/t0065-dir-iterator.sh | 111 ++++++++++++++++++ t/t2000-checkout-cache-clash.sh | 11 ++ 9 files changed, 433 insertions(+), 124 deletions(-) create mode 100644 t/helper/test-dir-iterator.c create mode 100755 t/t0065-dir-iterator.sh -- 2.7.4 (Apple Git-66)