Junio C Hamano | be3cfa8 | 2005-04-26 09:25:05 -0700 | [diff] [blame] | 1 | /* |
| 2 | * Copyright (C) 2005 Junio C Hamano |
| 3 | */ |
Junio C Hamano | 86436c2 | 2005-04-25 18:22:47 -0700 | [diff] [blame] | 4 | #include "cache.h" |
Junio C Hamano | 6fb737b | 2005-07-07 23:58:32 -0700 | [diff] [blame] | 5 | #include "quote.h" |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 6 | #include "commit.h" |
Junio C Hamano | 86436c2 | 2005-04-25 18:22:47 -0700 | [diff] [blame] | 7 | #include "diff.h" |
Junio C Hamano | 427dcb4 | 2005-05-21 02:39:09 -0700 | [diff] [blame] | 8 | #include "diffcore.h" |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 9 | #include "revision.h" |
Junio C Hamano | 1cfe773 | 2007-01-30 01:11:08 -0800 | [diff] [blame] | 10 | #include "cache-tree.h" |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 11 | #include "unpack-trees.h" |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 12 | #include "refs.h" |
Jens Lehmann | ee6fc51 | 2010-01-16 18:42:24 +0100 | [diff] [blame] | 13 | #include "submodule.h" |
Junio C Hamano | 427dcb4 | 2005-05-21 02:39:09 -0700 | [diff] [blame] | 14 | |
Junio C Hamano | b46f0b6 | 2005-05-04 01:45:24 -0700 | [diff] [blame] | 15 | /* |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 16 | * diff-files |
Junio C Hamano | b46f0b6 | 2005-05-04 01:45:24 -0700 | [diff] [blame] | 17 | */ |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 18 | |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 19 | /* |
Junio C Hamano | 1392a37 | 2008-05-03 17:04:42 -0700 | [diff] [blame] | 20 | * Has the work tree entity been removed? |
| 21 | * |
| 22 | * Return 1 if it was removed from the work tree, 0 if an entity to be |
| 23 | * compared with the cache entry ce still exists (the latter includes |
| 24 | * the case where a directory that is not a submodule repository |
| 25 | * exists for ce that is a submodule -- it is a submodule that is not |
| 26 | * checked out). Return negative for an error. |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 27 | */ |
Linus Torvalds | c40641b | 2008-05-09 09:21:07 -0700 | [diff] [blame] | 28 | static int check_removed(const struct cache_entry *ce, struct stat *st) |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 29 | { |
| 30 | if (lstat(ce->name, st) < 0) { |
| 31 | if (errno != ENOENT && errno != ENOTDIR) |
| 32 | return -1; |
| 33 | return 1; |
| 34 | } |
Kjetil Barvik | 5719989 | 2009-02-09 21:54:06 +0100 | [diff] [blame] | 35 | if (has_symlink_leading_path(ce->name, ce_namelen(ce))) |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 36 | return 1; |
| 37 | if (S_ISDIR(st->st_mode)) { |
| 38 | unsigned char sub[20]; |
Junio C Hamano | 1392a37 | 2008-05-03 17:04:42 -0700 | [diff] [blame] | 39 | |
| 40 | /* |
| 41 | * If ce is already a gitlink, we can have a plain |
| 42 | * directory (i.e. the submodule is not checked out), |
| 43 | * or a checked out submodule. Either case this is not |
| 44 | * a case where something was removed from the work tree, |
| 45 | * so we will return 0. |
| 46 | * |
| 47 | * Otherwise, if the directory is not a submodule |
| 48 | * repository, that means ce which was a blob turned into |
| 49 | * a directory --- the blob was removed! |
| 50 | */ |
| 51 | if (!S_ISGITLINK(ce->ce_mode) && |
| 52 | resolve_gitlink_ref(ce->name, "HEAD", sub)) |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 53 | return 1; |
| 54 | } |
| 55 | return 0; |
| 56 | } |
Johannes Schindelin | d516c2d | 2007-02-22 21:50:10 +0100 | [diff] [blame] | 57 | |
Jens Lehmann | ae6d5c1 | 2010-03-11 22:50:25 +0100 | [diff] [blame] | 58 | /* |
| 59 | * Has a file changed or has a submodule new commits or a dirty work tree? |
| 60 | * |
| 61 | * Return 1 when changes are detected, 0 otherwise. If the DIRTY_SUBMODULES |
| 62 | * option is set, the caller does not only want to know if a submodule is |
| 63 | * modified at all but wants to know all the conditions that are met (new |
| 64 | * commits, untracked content and/or modified content). |
| 65 | */ |
| 66 | static int match_stat_with_submodule(struct diff_options *diffopt, |
| 67 | struct cache_entry *ce, struct stat *st, |
| 68 | unsigned ce_option, unsigned *dirty_submodule) |
| 69 | { |
| 70 | int changed = ce_match_stat(ce, st, ce_option); |
Jens Lehmann | aee9c7d | 2010-08-06 00:39:25 +0200 | [diff] [blame] | 71 | if (S_ISGITLINK(ce->ce_mode)) { |
| 72 | unsigned orig_flags = diffopt->flags; |
| 73 | if (!DIFF_OPT_TST(diffopt, OVERRIDE_SUBMODULE_CONFIG)) |
| 74 | set_diffopt_flags_from_submodule_config(diffopt, ce->name); |
| 75 | if (DIFF_OPT_TST(diffopt, IGNORE_SUBMODULES)) |
| 76 | changed = 0; |
| 77 | else if (!DIFF_OPT_TST(diffopt, IGNORE_DIRTY_SUBMODULES) |
| 78 | && (!changed || DIFF_OPT_TST(diffopt, DIRTY_SUBMODULES))) |
| 79 | *dirty_submodule = is_submodule_modified(ce->name, DIFF_OPT_TST(diffopt, IGNORE_UNTRACKED_IN_SUBMODULES)); |
| 80 | diffopt->flags = orig_flags; |
Jens Lehmann | ae6d5c1 | 2010-03-11 22:50:25 +0100 | [diff] [blame] | 81 | } |
| 82 | return changed; |
| 83 | } |
| 84 | |
Junio C Hamano | 4bd5b7d | 2007-11-10 00:15:03 -0800 | [diff] [blame] | 85 | int run_diff_files(struct rev_info *revs, unsigned int option) |
Junio C Hamano | b46f0b6 | 2005-05-04 01:45:24 -0700 | [diff] [blame] | 86 | { |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 87 | int entries, i; |
| 88 | int diff_unmerged_stage = revs->max_count; |
Junio C Hamano | 4bd5b7d | 2007-11-10 00:15:03 -0800 | [diff] [blame] | 89 | int silent_on_removed = option & DIFF_SILENT_ON_REMOVED; |
Junio C Hamano | fb63d7f | 2007-11-09 18:22:52 -0800 | [diff] [blame] | 90 | unsigned ce_option = ((option & DIFF_RACY_IS_MODIFIED) |
| 91 | ? CE_MATCH_RACY_IS_DIRTY : 0); |
Junio C Hamano | 5c97558 | 2005-05-19 03:32:35 -0700 | [diff] [blame] | 92 | |
Junio C Hamano | a5a818e | 2008-08-18 20:08:09 -0700 | [diff] [blame] | 93 | diff_set_mnemonic_prefix(&revs->diffopt, "i/", "w/"); |
| 94 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 95 | if (diff_unmerged_stage < 0) |
| 96 | diff_unmerged_stage = 2; |
Junio C Hamano | b4e1e4a | 2007-02-09 18:51:40 -0800 | [diff] [blame] | 97 | entries = active_nr; |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 98 | for (i = 0; i < entries; i++) { |
Junio C Hamano | 427dcb4 | 2005-05-21 02:39:09 -0700 | [diff] [blame] | 99 | struct stat st; |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 100 | unsigned int oldmode, newmode; |
| 101 | struct cache_entry *ce = active_cache[i]; |
| 102 | int changed; |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 103 | unsigned dirty_submodule = 0; |
Junio C Hamano | f0c6b2a | 2005-05-27 15:56:38 -0700 | [diff] [blame] | 104 | |
Junio C Hamano | 90b1994 | 2009-05-23 01:15:35 -0700 | [diff] [blame] | 105 | if (DIFF_OPT_TST(&revs->diffopt, QUICK) && |
Pierre Habouzit | 8f67f8a | 2007-11-10 20:05:14 +0100 | [diff] [blame] | 106 | DIFF_OPT_TST(&revs->diffopt, HAS_CHANGES)) |
Junio C Hamano | 822cac0 | 2007-03-14 11:12:51 -0700 | [diff] [blame] | 107 | break; |
| 108 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 109 | if (!ce_path_match(ce, revs->prune_data)) |
Linus Torvalds | 8676eb4 | 2006-02-26 15:51:24 -0800 | [diff] [blame] | 110 | continue; |
Linus Torvalds | 8676eb4 | 2006-02-26 15:51:24 -0800 | [diff] [blame] | 111 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 112 | if (ce_stage(ce)) { |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 113 | struct combine_diff_path *dpath; |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 114 | int num_compare_stages = 0; |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 115 | size_t path_len; |
Linus Torvalds | 8676eb4 | 2006-02-26 15:51:24 -0800 | [diff] [blame] | 116 | |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 117 | path_len = ce_namelen(ce); |
| 118 | |
Junio C Hamano | 4fc970c | 2007-02-25 22:24:47 -0800 | [diff] [blame] | 119 | dpath = xmalloc(combine_diff_path_size(5, path_len)); |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 120 | dpath->path = (char *) &(dpath->parent[5]); |
| 121 | |
| 122 | dpath->next = NULL; |
| 123 | dpath->len = path_len; |
| 124 | memcpy(dpath->path, ce->name, path_len); |
| 125 | dpath->path[path_len] = '\0'; |
Junio C Hamano | a8e0d16 | 2006-08-23 13:57:23 -0700 | [diff] [blame] | 126 | hashclr(dpath->sha1); |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 127 | memset(&(dpath->parent[0]), 0, |
Junio C Hamano | 4fc970c | 2007-02-25 22:24:47 -0800 | [diff] [blame] | 128 | sizeof(struct combine_diff_parent)*5); |
| 129 | |
Linus Torvalds | c40641b | 2008-05-09 09:21:07 -0700 | [diff] [blame] | 130 | changed = check_removed(ce, &st); |
Junio C Hamano | f58dbf2 | 2008-03-30 17:30:08 -0700 | [diff] [blame] | 131 | if (!changed) |
| 132 | dpath->mode = ce_mode_from_stat(ce, st.st_mode); |
| 133 | else { |
| 134 | if (changed < 0) { |
Junio C Hamano | 4fc970c | 2007-02-25 22:24:47 -0800 | [diff] [blame] | 135 | perror(ce->name); |
| 136 | continue; |
| 137 | } |
| 138 | if (silent_on_removed) |
| 139 | continue; |
| 140 | } |
Junio C Hamano | be3cfa8 | 2005-04-26 09:25:05 -0700 | [diff] [blame] | 141 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 142 | while (i < entries) { |
| 143 | struct cache_entry *nce = active_cache[i]; |
| 144 | int stage; |
Junio C Hamano | be3cfa8 | 2005-04-26 09:25:05 -0700 | [diff] [blame] | 145 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 146 | if (strcmp(ce->name, nce->name)) |
| 147 | break; |
Junio C Hamano | be3cfa8 | 2005-04-26 09:25:05 -0700 | [diff] [blame] | 148 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 149 | /* Stage #2 (ours) is the first parent, |
| 150 | * stage #3 (theirs) is the second. |
| 151 | */ |
| 152 | stage = ce_stage(nce); |
| 153 | if (2 <= stage) { |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 154 | int mode = nce->ce_mode; |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 155 | num_compare_stages++; |
Shawn Pearce | e702496 | 2006-08-23 02:49:00 -0400 | [diff] [blame] | 156 | hashcpy(dpath->parent[stage-2].sha1, nce->sha1); |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 157 | dpath->parent[stage-2].mode = ce_mode_from_stat(nce, mode); |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 158 | dpath->parent[stage-2].status = |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 159 | DIFF_STATUS_MODIFIED; |
| 160 | } |
Junio C Hamano | cebff98 | 2006-03-25 23:12:17 -0800 | [diff] [blame] | 161 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 162 | /* diff against the proper unmerged stage */ |
| 163 | if (stage == diff_unmerged_stage) |
| 164 | ce = nce; |
| 165 | i++; |
H. Peter Anvin | 1b1480f | 2005-11-21 14:17:12 -0800 | [diff] [blame] | 166 | } |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 167 | /* |
| 168 | * Compensate for loop update |
| 169 | */ |
| 170 | i--; |
Junio C Hamano | 0e3994f | 2005-06-03 01:37:54 -0700 | [diff] [blame] | 171 | |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 172 | if (revs->combine_merges && num_compare_stages == 2) { |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 173 | show_combined_diff(dpath, 2, |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 174 | revs->dense_combined_merges, |
| 175 | revs); |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 176 | free(dpath); |
Junio C Hamano | 15d061b | 2005-05-27 15:55:55 -0700 | [diff] [blame] | 177 | continue; |
| 178 | } |
Florian Forster | b4b1550 | 2006-06-18 17:18:05 +0200 | [diff] [blame] | 179 | free(dpath); |
| 180 | dpath = NULL; |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 181 | |
| 182 | /* |
| 183 | * Show the diff for the 'ce' if we found the one |
| 184 | * from the desired stage. |
Junio C Hamano | 25d5ea4 | 2005-05-24 01:10:48 -0700 | [diff] [blame] | 185 | */ |
Junio C Hamano | e9c8409 | 2007-01-05 01:25:18 -0800 | [diff] [blame] | 186 | diff_unmerge(&revs->diffopt, ce->name, 0, null_sha1); |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 187 | if (ce_stage(ce) != diff_unmerged_stage) |
| 188 | continue; |
| 189 | } |
| 190 | |
Junio C Hamano | 125fd98 | 2010-01-24 00:10:20 -0800 | [diff] [blame] | 191 | if (ce_uptodate(ce) || ce_skip_worktree(ce)) |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 192 | continue; |
Junio C Hamano | f58dbf2 | 2008-03-30 17:30:08 -0700 | [diff] [blame] | 193 | |
Nguyễn Thái Ngọc Duy | 540e694 | 2009-08-11 22:43:59 +0700 | [diff] [blame] | 194 | /* If CE_VALID is set, don't look at workdir for file removal */ |
| 195 | changed = (ce->ce_flags & CE_VALID) ? 0 : check_removed(ce, &st); |
Junio C Hamano | f58dbf2 | 2008-03-30 17:30:08 -0700 | [diff] [blame] | 196 | if (changed) { |
| 197 | if (changed < 0) { |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 198 | perror(ce->name); |
| 199 | continue; |
Junio C Hamano | 25d5ea4 | 2005-05-24 01:10:48 -0700 | [diff] [blame] | 200 | } |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 201 | if (silent_on_removed) |
| 202 | continue; |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 203 | diff_addremove(&revs->diffopt, '-', ce->ce_mode, |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 204 | ce->sha1, ce->name, 0); |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 205 | continue; |
Junio C Hamano | bceafe7 | 2005-05-23 18:14:03 -0700 | [diff] [blame] | 206 | } |
Jens Lehmann | ae6d5c1 | 2010-03-11 22:50:25 +0100 | [diff] [blame] | 207 | changed = match_stat_with_submodule(&revs->diffopt, ce, &st, |
| 208 | ce_option, &dirty_submodule); |
Jens Lehmann | 85adbf2 | 2010-03-12 22:23:52 +0100 | [diff] [blame] | 209 | if (!changed && !dirty_submodule) { |
Junio C Hamano | 8fa2960 | 2008-03-30 12:39:25 -0700 | [diff] [blame] | 210 | ce_mark_uptodate(ce); |
| 211 | if (!DIFF_OPT_TST(&revs->diffopt, FIND_COPIES_HARDER)) |
| 212 | continue; |
| 213 | } |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 214 | oldmode = ce->ce_mode; |
| 215 | newmode = ce_mode_from_stat(ce, st.st_mode); |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 216 | diff_change(&revs->diffopt, oldmode, newmode, |
| 217 | ce->sha1, (changed ? null_sha1 : ce->sha1), |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 218 | ce->name, 0, dirty_submodule); |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 219 | |
Junio C Hamano | bceafe7 | 2005-05-23 18:14:03 -0700 | [diff] [blame] | 220 | } |
Junio C Hamano | 6973dca | 2006-04-21 23:57:45 -0700 | [diff] [blame] | 221 | diffcore_std(&revs->diffopt); |
| 222 | diff_flush(&revs->diffopt); |
| 223 | return 0; |
Junio C Hamano | bceafe7 | 2005-05-23 18:14:03 -0700 | [diff] [blame] | 224 | } |
| 225 | |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 226 | /* |
| 227 | * diff-index |
| 228 | */ |
| 229 | |
| 230 | /* A file entry went away or appeared */ |
| 231 | static void diff_index_show_file(struct rev_info *revs, |
| 232 | const char *prefix, |
| 233 | struct cache_entry *ce, |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 234 | const unsigned char *sha1, unsigned int mode, |
| 235 | unsigned dirty_submodule) |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 236 | { |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 237 | diff_addremove(&revs->diffopt, prefix[0], mode, |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 238 | sha1, ce->name, dirty_submodule); |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 239 | } |
| 240 | |
| 241 | static int get_stat_data(struct cache_entry *ce, |
Junio C Hamano | c8c16f2 | 2008-03-02 00:57:26 -0800 | [diff] [blame] | 242 | const unsigned char **sha1p, |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 243 | unsigned int *modep, |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 244 | int cached, int match_missing, |
Jens Lehmann | 4d34477 | 2010-01-23 17:37:26 +0100 | [diff] [blame] | 245 | unsigned *dirty_submodule, struct diff_options *diffopt) |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 246 | { |
Junio C Hamano | c8c16f2 | 2008-03-02 00:57:26 -0800 | [diff] [blame] | 247 | const unsigned char *sha1 = ce->sha1; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 248 | unsigned int mode = ce->ce_mode; |
| 249 | |
Linus Torvalds | 658dd48 | 2009-05-09 14:57:30 -0700 | [diff] [blame] | 250 | if (!cached && !ce_uptodate(ce)) { |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 251 | int changed; |
| 252 | struct stat st; |
Linus Torvalds | c40641b | 2008-05-09 09:21:07 -0700 | [diff] [blame] | 253 | changed = check_removed(ce, &st); |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 254 | if (changed < 0) |
| 255 | return -1; |
| 256 | else if (changed) { |
| 257 | if (match_missing) { |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 258 | *sha1p = sha1; |
| 259 | *modep = mode; |
| 260 | return 0; |
| 261 | } |
| 262 | return -1; |
| 263 | } |
Jens Lehmann | ae6d5c1 | 2010-03-11 22:50:25 +0100 | [diff] [blame] | 264 | changed = match_stat_with_submodule(diffopt, ce, &st, |
| 265 | 0, dirty_submodule); |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 266 | if (changed) { |
Junio C Hamano | 185c975 | 2007-02-16 22:43:48 -0800 | [diff] [blame] | 267 | mode = ce_mode_from_stat(ce, st.st_mode); |
Junio C Hamano | c8c16f2 | 2008-03-02 00:57:26 -0800 | [diff] [blame] | 268 | sha1 = null_sha1; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 269 | } |
| 270 | } |
| 271 | |
| 272 | *sha1p = sha1; |
| 273 | *modep = mode; |
| 274 | return 0; |
| 275 | } |
| 276 | |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 277 | static void show_new_file(struct rev_info *revs, |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 278 | struct cache_entry *new, |
| 279 | int cached, int match_missing) |
| 280 | { |
Junio C Hamano | c8c16f2 | 2008-03-02 00:57:26 -0800 | [diff] [blame] | 281 | const unsigned char *sha1; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 282 | unsigned int mode; |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 283 | unsigned dirty_submodule = 0; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 284 | |
Junio C Hamano | 948dd34 | 2008-03-30 17:29:48 -0700 | [diff] [blame] | 285 | /* |
| 286 | * New file in the index: it might actually be different in |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 287 | * the working copy. |
| 288 | */ |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 289 | if (get_stat_data(new, &sha1, &mode, cached, match_missing, |
Jens Lehmann | 4d34477 | 2010-01-23 17:37:26 +0100 | [diff] [blame] | 290 | &dirty_submodule, &revs->diffopt) < 0) |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 291 | return; |
| 292 | |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 293 | diff_index_show_file(revs, "+", new, sha1, mode, dirty_submodule); |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 294 | } |
| 295 | |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 296 | static int show_modified(struct rev_info *revs, |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 297 | struct cache_entry *old, |
| 298 | struct cache_entry *new, |
| 299 | int report_missing, |
| 300 | int cached, int match_missing) |
| 301 | { |
| 302 | unsigned int mode, oldmode; |
Junio C Hamano | c8c16f2 | 2008-03-02 00:57:26 -0800 | [diff] [blame] | 303 | const unsigned char *sha1; |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 304 | unsigned dirty_submodule = 0; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 305 | |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 306 | if (get_stat_data(new, &sha1, &mode, cached, match_missing, |
Jens Lehmann | 4d34477 | 2010-01-23 17:37:26 +0100 | [diff] [blame] | 307 | &dirty_submodule, &revs->diffopt) < 0) { |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 308 | if (report_missing) |
| 309 | diff_index_show_file(revs, "-", old, |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 310 | old->sha1, old->ce_mode, 0); |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 311 | return -1; |
| 312 | } |
| 313 | |
Paul Mackerras | cb2b9f5 | 2006-09-04 21:38:40 +1000 | [diff] [blame] | 314 | if (revs->combine_merges && !cached && |
| 315 | (hashcmp(sha1, old->sha1) || hashcmp(old->sha1, new->sha1))) { |
| 316 | struct combine_diff_path *p; |
| 317 | int pathlen = ce_namelen(new); |
| 318 | |
| 319 | p = xmalloc(combine_diff_path_size(2, pathlen)); |
| 320 | p->path = (char *) &p->parent[2]; |
| 321 | p->next = NULL; |
| 322 | p->len = pathlen; |
| 323 | memcpy(p->path, new->name, pathlen); |
| 324 | p->path[pathlen] = 0; |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 325 | p->mode = mode; |
Paul Mackerras | cb2b9f5 | 2006-09-04 21:38:40 +1000 | [diff] [blame] | 326 | hashclr(p->sha1); |
| 327 | memset(p->parent, 0, 2 * sizeof(struct combine_diff_parent)); |
| 328 | p->parent[0].status = DIFF_STATUS_MODIFIED; |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 329 | p->parent[0].mode = new->ce_mode; |
Paul Mackerras | cb2b9f5 | 2006-09-04 21:38:40 +1000 | [diff] [blame] | 330 | hashcpy(p->parent[0].sha1, new->sha1); |
| 331 | p->parent[1].status = DIFF_STATUS_MODIFIED; |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 332 | p->parent[1].mode = old->ce_mode; |
Paul Mackerras | cb2b9f5 | 2006-09-04 21:38:40 +1000 | [diff] [blame] | 333 | hashcpy(p->parent[1].sha1, old->sha1); |
| 334 | show_combined_diff(p, 2, revs->dense_combined_merges, revs); |
| 335 | free(p); |
| 336 | return 0; |
| 337 | } |
| 338 | |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 339 | oldmode = old->ce_mode; |
Jens Lehmann | 85adbf2 | 2010-03-12 22:23:52 +0100 | [diff] [blame] | 340 | if (mode == oldmode && !hashcmp(sha1, old->sha1) && !dirty_submodule && |
Pierre Habouzit | 8f67f8a | 2007-11-10 20:05:14 +0100 | [diff] [blame] | 341 | !DIFF_OPT_TST(&revs->diffopt, FIND_COPIES_HARDER)) |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 342 | return 0; |
| 343 | |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 344 | diff_change(&revs->diffopt, oldmode, mode, |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 345 | old->sha1, sha1, old->name, 0, dirty_submodule); |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 346 | return 0; |
| 347 | } |
| 348 | |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 349 | /* |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 350 | * This gets a mix of an existing index and a tree, one pathname entry |
| 351 | * at a time. The index entry may be a single stage-0 one, but it could |
| 352 | * also be multiple unmerged entries (in which case idx_pos/idx_nr will |
| 353 | * give you the position and number of entries in the index). |
| 354 | */ |
| 355 | static void do_oneway_diff(struct unpack_trees_options *o, |
| 356 | struct cache_entry *idx, |
Linus Torvalds | 34110cd | 2008-03-06 18:12:28 -0800 | [diff] [blame] | 357 | struct cache_entry *tree) |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 358 | { |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 359 | struct rev_info *revs = o->unpack_data; |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 360 | int match_missing, cached; |
Junio C Hamano | 5c21ac0 | 2006-04-22 03:58:04 -0700 | [diff] [blame] | 361 | |
Nguyễn Thái Ngọc Duy | 540e694 | 2009-08-11 22:43:59 +0700 | [diff] [blame] | 362 | /* if the entry is not checked out, don't examine work tree */ |
Nguyễn Thái Ngọc Duy | b4d1690 | 2009-08-20 20:46:58 +0700 | [diff] [blame] | 363 | cached = o->index_only || |
| 364 | (idx && ((idx->ce_flags & CE_VALID) || ce_skip_worktree(idx))); |
Junio C Hamano | a6080a0 | 2007-06-07 00:04:01 -0700 | [diff] [blame] | 365 | /* |
Junio C Hamano | 5c21ac0 | 2006-04-22 03:58:04 -0700 | [diff] [blame] | 366 | * Backward compatibility wart - "diff-index -m" does |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 367 | * not mean "do not ignore merges", but "match_missing". |
| 368 | * |
| 369 | * But with the revision flag parsing, that's found in |
| 370 | * "!revs->ignore_merges". |
Junio C Hamano | 5c21ac0 | 2006-04-22 03:58:04 -0700 | [diff] [blame] | 371 | */ |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 372 | match_missing = !revs->ignore_merges; |
| 373 | |
| 374 | if (cached && idx && ce_stage(idx)) { |
Junio C Hamano | 29796c6 | 2009-08-04 16:25:40 -0700 | [diff] [blame] | 375 | diff_unmerge(&revs->diffopt, idx->name, idx->ce_mode, |
| 376 | idx->sha1); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 377 | return; |
| 378 | } |
| 379 | |
| 380 | /* |
| 381 | * Something added to the tree? |
| 382 | */ |
| 383 | if (!tree) { |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 384 | show_new_file(revs, idx, cached, match_missing); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 385 | return; |
| 386 | } |
| 387 | |
| 388 | /* |
| 389 | * Something removed from the tree? |
| 390 | */ |
| 391 | if (!idx) { |
Jens Lehmann | e3d42c4 | 2010-01-18 21:26:18 +0100 | [diff] [blame] | 392 | diff_index_show_file(revs, "-", tree, tree->sha1, tree->ce_mode, 0); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 393 | return; |
| 394 | } |
| 395 | |
| 396 | /* Show difference between old and new */ |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 397 | show_modified(revs, tree, idx, 1, cached, match_missing); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 398 | } |
| 399 | |
| 400 | /* |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 401 | * The unpack_trees() interface is designed for merging, so |
| 402 | * the different source entries are designed primarily for |
| 403 | * the source trees, with the old index being really mainly |
| 404 | * used for being replaced by the result. |
| 405 | * |
| 406 | * For diffing, the index is more important, and we only have a |
| 407 | * single tree. |
| 408 | * |
Junio C Hamano | da8ba5e | 2009-09-17 22:12:17 -0700 | [diff] [blame] | 409 | * We're supposed to advance o->pos to skip what we have already processed. |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 410 | * |
| 411 | * This wrapper makes it all more readable, and takes care of all |
| 412 | * the fairly complex unpack_trees() semantic requirements, including |
| 413 | * the skipping, the path matching, the type conflict cases etc. |
| 414 | */ |
Linus Torvalds | 34110cd | 2008-03-06 18:12:28 -0800 | [diff] [blame] | 415 | static int oneway_diff(struct cache_entry **src, struct unpack_trees_options *o) |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 416 | { |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 417 | struct cache_entry *idx = src[0]; |
| 418 | struct cache_entry *tree = src[1]; |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 419 | struct rev_info *revs = o->unpack_data; |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 420 | |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 421 | /* |
| 422 | * Unpack-trees generates a DF/conflict entry if |
| 423 | * there was a directory in the index and a tree |
| 424 | * in the tree. From a diff standpoint, that's a |
| 425 | * delete of the tree and a create of the file. |
| 426 | */ |
| 427 | if (tree == o->df_conflict_entry) |
| 428 | tree = NULL; |
| 429 | |
| 430 | if (ce_path_match(idx ? idx : tree, revs->prune_data)) |
Linus Torvalds | 34110cd | 2008-03-06 18:12:28 -0800 | [diff] [blame] | 431 | do_oneway_diff(o, idx, tree); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 432 | |
Linus Torvalds | 34110cd | 2008-03-06 18:12:28 -0800 | [diff] [blame] | 433 | return 0; |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 434 | } |
| 435 | |
| 436 | int run_diff_index(struct rev_info *revs, int cached) |
| 437 | { |
| 438 | struct object *ent; |
| 439 | struct tree *tree; |
| 440 | const char *tree_name; |
| 441 | struct unpack_trees_options opts; |
| 442 | struct tree_desc t; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 443 | |
Linus Torvalds | 1f1e895 | 2006-06-19 17:42:35 -0700 | [diff] [blame] | 444 | ent = revs->pending.objects[0].item; |
| 445 | tree_name = revs->pending.objects[0].name; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 446 | tree = parse_tree_indirect(ent->sha1); |
| 447 | if (!tree) |
| 448 | return error("bad tree object %s", tree_name); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 449 | |
| 450 | memset(&opts, 0, sizeof(opts)); |
| 451 | opts.head_idx = 1; |
| 452 | opts.index_only = cached; |
Junio C Hamano | a0919ce | 2009-05-22 23:14:25 -0700 | [diff] [blame] | 453 | opts.diff_index_cached = (cached && |
| 454 | !DIFF_OPT_TST(&revs->diffopt, FIND_COPIES_HARDER)); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 455 | opts.merge = 1; |
| 456 | opts.fn = oneway_diff; |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 457 | opts.unpack_data = revs; |
Linus Torvalds | 34110cd | 2008-03-06 18:12:28 -0800 | [diff] [blame] | 458 | opts.src_index = &the_index; |
| 459 | opts.dst_index = NULL; |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 460 | |
| 461 | init_tree_desc(&t, tree->buffer, tree->size); |
Daniel Barkalow | 203a2fe | 2008-02-07 11:39:48 -0500 | [diff] [blame] | 462 | if (unpack_trees(1, &t, &opts)) |
| 463 | exit(128); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 464 | |
Junio C Hamano | a5a818e | 2008-08-18 20:08:09 -0700 | [diff] [blame] | 465 | diff_set_mnemonic_prefix(&revs->diffopt, "c/", cached ? "i/" : "w/"); |
Junio C Hamano | 730f728 | 2009-09-20 00:03:39 -0700 | [diff] [blame] | 466 | diffcore_fix_diff_index(&revs->diffopt); |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 467 | diffcore_std(&revs->diffopt); |
| 468 | diff_flush(&revs->diffopt); |
Linus Torvalds | d1f2d7e | 2008-01-19 17:27:12 -0800 | [diff] [blame] | 469 | return 0; |
Junio C Hamano | e09ad6e | 2006-04-22 02:43:00 -0700 | [diff] [blame] | 470 | } |
Junio C Hamano | 1cfe773 | 2007-01-30 01:11:08 -0800 | [diff] [blame] | 471 | |
| 472 | int do_diff_cache(const unsigned char *tree_sha1, struct diff_options *opt) |
| 473 | { |
| 474 | struct tree *tree; |
| 475 | struct rev_info revs; |
| 476 | int i; |
| 477 | struct cache_entry **dst; |
| 478 | struct cache_entry *last = NULL; |
Johannes Schindelin | 204ce97 | 2008-01-20 15:19:56 +0000 | [diff] [blame] | 479 | struct unpack_trees_options opts; |
| 480 | struct tree_desc t; |
Junio C Hamano | 1cfe773 | 2007-01-30 01:11:08 -0800 | [diff] [blame] | 481 | |
| 482 | /* |
| 483 | * This is used by git-blame to run diff-cache internally; |
| 484 | * it potentially needs to repeatedly run this, so we will |
| 485 | * start by removing the higher order entries the last round |
| 486 | * left behind. |
| 487 | */ |
| 488 | dst = active_cache; |
| 489 | for (i = 0; i < active_nr; i++) { |
| 490 | struct cache_entry *ce = active_cache[i]; |
| 491 | if (ce_stage(ce)) { |
| 492 | if (last && !strcmp(ce->name, last->name)) |
| 493 | continue; |
| 494 | cache_tree_invalidate_path(active_cache_tree, |
| 495 | ce->name); |
| 496 | last = ce; |
Linus Torvalds | 7a51ed6 | 2008-01-14 16:03:17 -0800 | [diff] [blame] | 497 | ce->ce_flags |= CE_REMOVE; |
Junio C Hamano | 1cfe773 | 2007-01-30 01:11:08 -0800 | [diff] [blame] | 498 | } |
| 499 | *dst++ = ce; |
| 500 | } |
| 501 | active_nr = dst - active_cache; |
| 502 | |
| 503 | init_revisions(&revs, NULL); |
| 504 | revs.prune_data = opt->paths; |
| 505 | tree = parse_tree_indirect(tree_sha1); |
| 506 | if (!tree) |
| 507 | die("bad tree object %s", sha1_to_hex(tree_sha1)); |
Johannes Schindelin | 204ce97 | 2008-01-20 15:19:56 +0000 | [diff] [blame] | 508 | |
| 509 | memset(&opts, 0, sizeof(opts)); |
| 510 | opts.head_idx = 1; |
| 511 | opts.index_only = 1; |
Junio C Hamano | a0919ce | 2009-05-22 23:14:25 -0700 | [diff] [blame] | 512 | opts.diff_index_cached = !DIFF_OPT_TST(opt, FIND_COPIES_HARDER); |
Johannes Schindelin | 204ce97 | 2008-01-20 15:19:56 +0000 | [diff] [blame] | 513 | opts.merge = 1; |
| 514 | opts.fn = oneway_diff; |
Kjetil Barvik | ff7e6aa | 2009-01-11 19:36:42 +0100 | [diff] [blame] | 515 | opts.unpack_data = &revs; |
Linus Torvalds | 34110cd | 2008-03-06 18:12:28 -0800 | [diff] [blame] | 516 | opts.src_index = &the_index; |
| 517 | opts.dst_index = &the_index; |
Johannes Schindelin | 204ce97 | 2008-01-20 15:19:56 +0000 | [diff] [blame] | 518 | |
| 519 | init_tree_desc(&t, tree->buffer, tree->size); |
Daniel Barkalow | 203a2fe | 2008-02-07 11:39:48 -0500 | [diff] [blame] | 520 | if (unpack_trees(1, &t, &opts)) |
| 521 | exit(128); |
Johannes Schindelin | 204ce97 | 2008-01-20 15:19:56 +0000 | [diff] [blame] | 522 | return 0; |
Junio C Hamano | 1cfe773 | 2007-01-30 01:11:08 -0800 | [diff] [blame] | 523 | } |
Stephan Beyer | 75f3ff2 | 2009-02-10 15:30:35 +0100 | [diff] [blame] | 524 | |
| 525 | int index_differs_from(const char *def, int diff_flags) |
| 526 | { |
| 527 | struct rev_info rev; |
Junio C Hamano | 32962c9 | 2010-03-08 22:58:09 -0800 | [diff] [blame] | 528 | struct setup_revision_opt opt; |
Stephan Beyer | 75f3ff2 | 2009-02-10 15:30:35 +0100 | [diff] [blame] | 529 | |
| 530 | init_revisions(&rev, NULL); |
Junio C Hamano | 32962c9 | 2010-03-08 22:58:09 -0800 | [diff] [blame] | 531 | memset(&opt, 0, sizeof(opt)); |
| 532 | opt.def = def; |
| 533 | setup_revisions(0, NULL, &rev, &opt); |
Junio C Hamano | 90b1994 | 2009-05-23 01:15:35 -0700 | [diff] [blame] | 534 | DIFF_OPT_SET(&rev.diffopt, QUICK); |
Stephan Beyer | 75f3ff2 | 2009-02-10 15:30:35 +0100 | [diff] [blame] | 535 | DIFF_OPT_SET(&rev.diffopt, EXIT_WITH_STATUS); |
| 536 | rev.diffopt.flags |= diff_flags; |
| 537 | run_diff_index(&rev, 1); |
| 538 | if (rev.pending.alloc) |
| 539 | free(rev.pending.objects); |
| 540 | return (DIFF_OPT_TST(&rev.diffopt, HAS_CHANGES) != 0); |
| 541 | } |