Patch Set 7: I would prefer that you didn't submit this

(1 inline comment)

I wonder if this change is needed.

- If I understand it correctly the method scan(TreeWalk walk, boolean includeTrees) with includeTrees == true behaves the same as if the walk was set to recursive before the method call. What is the usecase then for the new method?

- In your EGit changes I saw that you added a usage in I7ddb21e7ff54333dd6d7ace3209bbcf83da2b219 but you removed it again in the dependent change I6e6f48fe16fa7a072905c3fcc01e9a4019470543.