[PATCH 2/3] daemon: cleanup: replace loop with if
- From
René Scharfe <rene.scharfe@lsrfire.ath.cx>
- Date
- Dec 26, 2008, 10:01 UTC
- Message-ID
- <1230285717.6728.40.camel@ubuntu.ubuntu-domain>
- In-Reply-To
- <1230284785.6728.26.camel@ubuntu.ubuntu-domain>
Replace a loop around an enter_repo() call, which was used to retry a single time with a different parameter in case the first call fails, with two calls and an if. This is shorter and cleaner.
Signed-off-by: Rene Scharfe <rene.scharfe@lsrfire.ath.cx> --- daemon.c | 18 +++++------------- 1 files changed, 5 insertions(+), 13 deletions(-)
diff --git a/daemon.c b/daemon.c index 8c317be..4468fb9 100644 --- a/daemon.c +++ b/daemon.c @@ -150,7 +150,6 @@ static char *path_ok(char *directory) { static char rpath[PATH_MAX]; static char interp_path[PATH_MAX]; - int retried_path = 0; char *path; char *dir; @@ -219,22 +218,15 @@ static char *path_ok(char *directory) dir = rpath; } - do { - path = enter_repo(dir, strict_paths); - if (path) - break; - + path = enter_repo(dir, strict_paths); + if (!path && base_path && base_path_relaxed) { /* * if we fail and base_path_relaxed is enabled, try without * prefixing the base path */ - if (base_path && base_path_relaxed && !retried_path) { - dir = directory; - retried_path = 1; - continue; - } - break; - } while (1); + dir = directory; + path = enter_repo(dir, strict_paths); + } if (!path) { logerror("'%s': unable to chdir or not a git archive", dir);
-- 1.6.1