Remove arbitrary iteration limit (#745)

The limit of 10000 could be reached for very long histories and
produce false failure conditions.
Given that all git DAGs contain a finite number of commits
the loop should always terminate anyway.

Change-Id: for2loop

Author: christian-schilling

src/filter/mod.rs

@@ -228,7 +228,7 @@ pub fn apply_to_commit(
     transaction: &cache::Transaction,
 ) -> JoshResult<git2::Oid> {
     let filter = opt::optimize(filter);
-    for _ in 0..10000 {
+    loop {
         let filtered = apply_to_commit2(&to_op(filter), commit, transaction)?;
 
         if let Some(id) = filtered {
@@ -239,8 +239,6 @@ pub fn apply_to_commit(
             history::walk2(f, i, transaction)?;
         }
     }
-
-    Err(josh_error("apply_to_commit did not finish"))
 }
 
 pub fn apply_to_commit3(