diff options
author | Ben Straub <bs@github.com> | 2013-12-05 14:47:04 -0800 |
---|---|---|
committer | Ben Straub <bs@github.com> | 2013-12-05 14:47:04 -0800 |
commit | 628e92cdb31cb3561549671ea5346a347f2addcd (patch) | |
tree | 81773bec0b395fe1d91dd27124ef4cae55b0f675 /src/diff_tform.c | |
parent | c56c6d694563fdd28cd00ed246f35349522b836e (diff) | |
download | libgit2-628e92cdb31cb3561549671ea5346a347f2addcd.tar.gz |
Don't use weird return codes
Diffstat (limited to 'src/diff_tform.c')
-rw-r--r-- | src/diff_tform.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/src/diff_tform.c b/src/diff_tform.c index 6f29f25da..0c9f961f2 100644 --- a/src/diff_tform.c +++ b/src/diff_tform.c @@ -293,8 +293,9 @@ static int normalize_find_opts( if (git_config_get_string(&val, cfg, "diff.renames") < 0) giterr_clear(); else if (val) { - if (!strcasecmp(val, "false")) - return GIT_PASSTHROUGH; + int boolval; + if (!git__parse_bool(&boolval, val) && !boolval) + opts->flags = 0; else if (!strcasecmp(val, "copies") || !strcasecmp(val, "copy")) opts->flags = GIT_DIFF_FIND_RENAMES | GIT_DIFF_FIND_COPIES; else @@ -834,7 +835,11 @@ int git_diff_find_similar( git_diff_file swap; if ((error = normalize_find_opts(diff, &opts, given_opts)) < 0) - return (error == GIT_PASSTHROUGH) ? 0 : error; + return error; + + /* No flags set; nothing to do */ + if ((opts.flags & GIT_DIFF_FIND_ALL) == 0) + return 0; num_deltas = diff->deltas.length; |