Commit 16a666d3d421fcbffabc861646b02e22c63d9ad9
2013-01-02T17:05:54
Fix workdir notifications and removals The notifications were broken from the various iterations over this code and were not returning working dir item data correctly. Also, workdir items that were alphabetically after the last item in diff were not being processed.
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91
diff --git a/src/checkout.c b/src/checkout.c
index 76119c6..962250e 100644
--- a/src/checkout.c
+++ b/src/checkout.c
@@ -209,9 +209,9 @@ static int checkout_notify(
checkout_data *data,
git_checkout_notify_t why,
const git_diff_delta *delta,
- const git_index_entry *baseitem)
+ const git_index_entry *wditem)
{
- git_diff_file basefile;
+ git_diff_file wdfile;
const git_diff_file *baseline = NULL, *target = NULL, *workdir = NULL;
if (!data->opts.notify_cb)
@@ -220,44 +220,41 @@ static int checkout_notify(
if ((why & data->opts.notify_flags) == 0)
return 0;
- if (baseitem) {
- memset(&basefile, 0, sizeof(basefile));
+ if (wditem) {
+ memset(&wdfile, 0, sizeof(wdfile));
- git_oid_cpy(&basefile.oid, &baseitem->oid);
- basefile.path = baseitem->path;
- basefile.size = baseitem->file_size;
- basefile.flags = GIT_DIFF_FILE_VALID_OID;
- basefile.mode = baseitem->mode;
+ git_oid_cpy(&wdfile.oid, &wditem->oid);
+ wdfile.path = wditem->path;
+ wdfile.size = wditem->file_size;
+ wdfile.flags = GIT_DIFF_FILE_VALID_OID;
+ wdfile.mode = wditem->mode;
- baseline = &basefile;
+ workdir = &wdfile;
}
- if ((why & GIT_CHECKOUT__NOTIFY_CONFLICT_TREE) != 0) {
- /* baseitem is a blob that conflicts with a tree in the workdir */
- } else {
+ if (delta) {
switch (delta->status) {
case GIT_DELTA_UNMODIFIED:
case GIT_DELTA_MODIFIED:
case GIT_DELTA_TYPECHANGE:
default:
- target = &delta->old_file;
- workdir = &delta->new_file;
+ baseline = &delta->old_file;
+ target = &delta->new_file;
break;
case GIT_DELTA_ADDED:
case GIT_DELTA_IGNORED:
case GIT_DELTA_UNTRACKED:
- workdir = &delta->new_file;
+ target = &delta->new_file;
break;
case GIT_DELTA_DELETED:
- target = &delta->old_file;
+ baseline = &delta->old_file;
break;
}
}
return data->opts.notify_cb(
- why, delta->old_file.path,
- baseline, target, workdir,
- data->opts.notify_payload);
+ why, delta ? delta->old_file.path : wditem->path,
+ baseline, target, workdir, data->opts.notify_payload);
}
static bool checkout_is_workdir_modified(
@@ -653,6 +650,14 @@ static int checkout_get_actions(
counts[CHECKOUT_ACTION__CONFLICT]++;
}
+ while (wditem != NULL) {
+ error = checkout_action_wd_only(data, workdir, wditem, &pathspec);
+ if (!error)
+ error = git_iterator_advance(workdir, &wditem);
+ if (error < 0)
+ goto fail;
+ }
+
counts[CHECKOUT_ACTION__REMOVE] += data->removes.length;
if (counts[CHECKOUT_ACTION__CONFLICT] > 0 &&