author | Lars Hjemli <hjemli@gmail.com> | 2008-12-26 10:02:02 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2008-12-26 10:02:02 (UTC) |
commit | 0edf76078e6a36ba502e6ffb97021166ea459a7f (patch) (unidiff) | |
tree | 7c3392f15eaa75855be9777da59f7f655c93af92 | |
parent | 11456a60deab19f5e3a1d191bdf48adfba9195e4 (diff) | |
download | cgit-0edf76078e6a36ba502e6ffb97021166ea459a7f.zip cgit-0edf76078e6a36ba502e6ffb97021166ea459a7f.tar.gz cgit-0edf76078e6a36ba502e6ffb97021166ea459a7f.tar.bz2 |
shared.c: future-proof usage of git diff-structures
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | shared.c | 4 |
1 files changed, 3 insertions, 1 deletions
@@ -238,68 +238,70 @@ int filediff_cb(void *priv, mmbuffer_t *mb, int nbuf) | |||
238 | if (!diffbuf) { | 238 | if (!diffbuf) { |
239 | ((linediff_fn)priv)(mb[i].ptr, mb[i].size); | 239 | ((linediff_fn)priv)(mb[i].ptr, mb[i].size); |
240 | continue; | 240 | continue; |
241 | } | 241 | } |
242 | diffbuf = xrealloc(diffbuf, buflen + mb[i].size); | 242 | diffbuf = xrealloc(diffbuf, buflen + mb[i].size); |
243 | memcpy(diffbuf + buflen, mb[i].ptr, mb[i].size); | 243 | memcpy(diffbuf + buflen, mb[i].ptr, mb[i].size); |
244 | ((linediff_fn)priv)(diffbuf, buflen + mb[i].size); | 244 | ((linediff_fn)priv)(diffbuf, buflen + mb[i].size); |
245 | free(diffbuf); | 245 | free(diffbuf); |
246 | diffbuf = NULL; | 246 | diffbuf = NULL; |
247 | buflen = 0; | 247 | buflen = 0; |
248 | } | 248 | } |
249 | if (diffbuf) { | 249 | if (diffbuf) { |
250 | ((linediff_fn)priv)(diffbuf, buflen); | 250 | ((linediff_fn)priv)(diffbuf, buflen); |
251 | free(diffbuf); | 251 | free(diffbuf); |
252 | diffbuf = NULL; | 252 | diffbuf = NULL; |
253 | buflen = 0; | 253 | buflen = 0; |
254 | } | 254 | } |
255 | return 0; | 255 | return 0; |
256 | } | 256 | } |
257 | 257 | ||
258 | int cgit_diff_files(const unsigned char *old_sha1, | 258 | int cgit_diff_files(const unsigned char *old_sha1, |
259 | const unsigned char *new_sha1, | 259 | const unsigned char *new_sha1, |
260 | linediff_fn fn) | 260 | linediff_fn fn) |
261 | { | 261 | { |
262 | mmfile_t file1, file2; | 262 | mmfile_t file1, file2; |
263 | xpparam_t diff_params; | 263 | xpparam_t diff_params; |
264 | xdemitconf_t emit_params; | 264 | xdemitconf_t emit_params; |
265 | xdemitcb_t emit_cb; | 265 | xdemitcb_t emit_cb; |
266 | 266 | ||
267 | if (!load_mmfile(&file1, old_sha1) || !load_mmfile(&file2, new_sha1)) | 267 | if (!load_mmfile(&file1, old_sha1) || !load_mmfile(&file2, new_sha1)) |
268 | return 1; | 268 | return 1; |
269 | 269 | ||
270 | memset(&diff_params, 0, sizeof(diff_params)); | ||
271 | memset(&emit_params, 0, sizeof(emit_params)); | ||
272 | memset(&emit_cb, 0, sizeof(emit_cb)); | ||
270 | diff_params.flags = XDF_NEED_MINIMAL; | 273 | diff_params.flags = XDF_NEED_MINIMAL; |
271 | emit_params.ctxlen = 3; | 274 | emit_params.ctxlen = 3; |
272 | emit_params.flags = XDL_EMIT_FUNCNAMES; | 275 | emit_params.flags = XDL_EMIT_FUNCNAMES; |
273 | emit_params.find_func = NULL; | ||
274 | emit_cb.outf = filediff_cb; | 276 | emit_cb.outf = filediff_cb; |
275 | emit_cb.priv = fn; | 277 | emit_cb.priv = fn; |
276 | xdl_diff(&file1, &file2, &diff_params, &emit_params, &emit_cb); | 278 | xdl_diff(&file1, &file2, &diff_params, &emit_params, &emit_cb); |
277 | return 0; | 279 | return 0; |
278 | } | 280 | } |
279 | 281 | ||
280 | void cgit_diff_tree(const unsigned char *old_sha1, | 282 | void cgit_diff_tree(const unsigned char *old_sha1, |
281 | const unsigned char *new_sha1, | 283 | const unsigned char *new_sha1, |
282 | filepair_fn fn, const char *prefix) | 284 | filepair_fn fn, const char *prefix) |
283 | { | 285 | { |
284 | struct diff_options opt; | 286 | struct diff_options opt; |
285 | int ret; | 287 | int ret; |
286 | int prefixlen; | 288 | int prefixlen; |
287 | 289 | ||
288 | diff_setup(&opt); | 290 | diff_setup(&opt); |
289 | opt.output_format = DIFF_FORMAT_CALLBACK; | 291 | opt.output_format = DIFF_FORMAT_CALLBACK; |
290 | opt.detect_rename = 1; | 292 | opt.detect_rename = 1; |
291 | opt.rename_limit = ctx.cfg.renamelimit; | 293 | opt.rename_limit = ctx.cfg.renamelimit; |
292 | DIFF_OPT_SET(&opt, RECURSIVE); | 294 | DIFF_OPT_SET(&opt, RECURSIVE); |
293 | opt.format_callback = cgit_diff_tree_cb; | 295 | opt.format_callback = cgit_diff_tree_cb; |
294 | opt.format_callback_data = fn; | 296 | opt.format_callback_data = fn; |
295 | if (prefix) { | 297 | if (prefix) { |
296 | opt.nr_paths = 1; | 298 | opt.nr_paths = 1; |
297 | opt.paths = &prefix; | 299 | opt.paths = &prefix; |
298 | prefixlen = strlen(prefix); | 300 | prefixlen = strlen(prefix); |
299 | opt.pathlens = &prefixlen; | 301 | opt.pathlens = &prefixlen; |
300 | } | 302 | } |
301 | diff_setup_done(&opt); | 303 | diff_setup_done(&opt); |
302 | 304 | ||
303 | if (old_sha1 && !is_null_sha1(old_sha1)) | 305 | if (old_sha1 && !is_null_sha1(old_sha1)) |
304 | ret = diff_tree_sha1(old_sha1, new_sha1, "", &opt); | 306 | ret = diff_tree_sha1(old_sha1, new_sha1, "", &opt); |
305 | else | 307 | else |