]> source.dussan.org Git - jgit.git/commit
Fix PathFilterGroup not to throw StopWalkException too early 97/11597/3
authorRobin Rosenberg <robin.rosenberg@dewire.com>
Tue, 2 Apr 2013 07:00:58 +0000 (09:00 +0200)
committerGerrit Code Review @ Eclipse.org <gerrit@eclipse.org>
Wed, 3 Apr 2013 18:07:23 +0000 (14:07 -0400)
commitc9a94dc1eeabeda212ed0b2eab0afdd67331b848
treeda8e1dd6890094d537799761e59cd8e41aa55342
parent65027d8bb429581635f51d0c588493e5c458cc25
Fix PathFilterGroup not to throw StopWalkException too early

Due to the Git internal sort order a directory is sorted as if it ended
with a '/', this means that the path filter didn't set the last possible
matching entry to the correct value. In the reported issue we had the
following filters.

org.eclipse.jgit.console
org.eclipse.jgit

As an optimization we throw a StopWalkException when the walked tree
passes the last possible filter, which was this:
org.eclipse.jgit.console

Due to the git sorting order, the tree was processed in this order:
org.eclipse.jgit.console
org.eclipse.jgit.test
org.eclipse.jgit

At org.eclipse.jgit.test we threw the StopWalkException preventing the
walk from completing successfully.

A correct last possible match should be:
org.eclipse.jgit/

For simplicit we define it as:
org/eclipse/jgit/

This filter would be the maximum if we also had e.g. org and org.eclipse
in the filter, but that would require more work so we simply replace all
characters lower than '/' by a slash.

We believe the possible extra walking does not not warrant the extra
analysis.

Bug: 362430
Change-Id: I4869019ea57ca07d4dff6bfa8e81725f56596d9f
org.eclipse.jgit.test/tst/org/eclipse/jgit/treewalk/filter/PathFilterGroupTest.java
org.eclipse.jgit/src/org/eclipse/jgit/treewalk/filter/PathFilterGroup.java