author | Lars Hjemli <hjemli@gmail.com> | 2007-09-04 09:04:47 (UTC) |
---|---|---|
committer | Lars Hjemli <hjemli@gmail.com> | 2007-09-04 09:04:47 (UTC) |
commit | 0de21a8d4176f15b4185a2e083787eeb05d95a77 (patch) (unidiff) | |
tree | 9504f1693e2ad921e687b736c3a38c0fe9e7a2e0 /shared.c | |
parent | ff13396926f3352d9e29c0663154918297dc5af7 (diff) | |
download | cgit-0de21a8d4176f15b4185a2e083787eeb05d95a77.zip cgit-0de21a8d4176f15b4185a2e083787eeb05d95a77.tar.gz cgit-0de21a8d4176f15b4185a2e083787eeb05d95a77.tar.bz2 |
Set xdemitconf_t.findfunc=NULL
This function-pointer was introduced in git v1.5.3-rc0-42-gf258475. It
needs to have a value, and setting it to NULL triggers the old behaviour
for selecting hunk headers.
Signed-off-by: Lars Hjemli <hjemli@gmail.com>
-rw-r--r-- | shared.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -184,229 +184,230 @@ void cgit_global_config_cb(const char *name, const char *value) | |||
184 | cgit_agefile = xstrdup(value); | 184 | cgit_agefile = xstrdup(value); |
185 | else if (!strcmp(name, "repo.group")) | 185 | else if (!strcmp(name, "repo.group")) |
186 | cgit_repo_group = xstrdup(value); | 186 | cgit_repo_group = xstrdup(value); |
187 | else if (!strcmp(name, "repo.url")) | 187 | else if (!strcmp(name, "repo.url")) |
188 | cgit_repo = add_repo(value); | 188 | cgit_repo = add_repo(value); |
189 | else if (!strcmp(name, "repo.name")) | 189 | else if (!strcmp(name, "repo.name")) |
190 | cgit_repo->name = xstrdup(value); | 190 | cgit_repo->name = xstrdup(value); |
191 | else if (cgit_repo && !strcmp(name, "repo.path")) | 191 | else if (cgit_repo && !strcmp(name, "repo.path")) |
192 | cgit_repo->path = xstrdup(value); | 192 | cgit_repo->path = xstrdup(value); |
193 | else if (cgit_repo && !strcmp(name, "repo.desc")) | 193 | else if (cgit_repo && !strcmp(name, "repo.desc")) |
194 | cgit_repo->desc = xstrdup(value); | 194 | cgit_repo->desc = xstrdup(value); |
195 | else if (cgit_repo && !strcmp(name, "repo.owner")) | 195 | else if (cgit_repo && !strcmp(name, "repo.owner")) |
196 | cgit_repo->owner = xstrdup(value); | 196 | cgit_repo->owner = xstrdup(value); |
197 | else if (cgit_repo && !strcmp(name, "repo.defbranch")) | 197 | else if (cgit_repo && !strcmp(name, "repo.defbranch")) |
198 | cgit_repo->defbranch = xstrdup(value); | 198 | cgit_repo->defbranch = xstrdup(value); |
199 | else if (cgit_repo && !strcmp(name, "repo.snapshots")) | 199 | else if (cgit_repo && !strcmp(name, "repo.snapshots")) |
200 | cgit_repo->snapshots = cgit_snapshots & cgit_parse_snapshots_mask(value); /* XXX: &? */ | 200 | cgit_repo->snapshots = cgit_snapshots & cgit_parse_snapshots_mask(value); /* XXX: &? */ |
201 | else if (cgit_repo && !strcmp(name, "repo.enable-log-filecount")) | 201 | else if (cgit_repo && !strcmp(name, "repo.enable-log-filecount")) |
202 | cgit_repo->enable_log_filecount = cgit_enable_log_filecount * atoi(value); | 202 | cgit_repo->enable_log_filecount = cgit_enable_log_filecount * atoi(value); |
203 | else if (cgit_repo && !strcmp(name, "repo.enable-log-linecount")) | 203 | else if (cgit_repo && !strcmp(name, "repo.enable-log-linecount")) |
204 | cgit_repo->enable_log_linecount = cgit_enable_log_linecount * atoi(value); | 204 | cgit_repo->enable_log_linecount = cgit_enable_log_linecount * atoi(value); |
205 | else if (cgit_repo && !strcmp(name, "repo.module-link")) | 205 | else if (cgit_repo && !strcmp(name, "repo.module-link")) |
206 | cgit_repo->module_link= xstrdup(value); | 206 | cgit_repo->module_link= xstrdup(value); |
207 | else if (cgit_repo && !strcmp(name, "repo.readme") && value != NULL) { | 207 | else if (cgit_repo && !strcmp(name, "repo.readme") && value != NULL) { |
208 | if (*value == '/') | 208 | if (*value == '/') |
209 | cgit_repo->readme = xstrdup(value); | 209 | cgit_repo->readme = xstrdup(value); |
210 | else | 210 | else |
211 | cgit_repo->readme = xstrdup(fmt("%s/%s", cgit_repo->path, value)); | 211 | cgit_repo->readme = xstrdup(fmt("%s/%s", cgit_repo->path, value)); |
212 | } else if (!strcmp(name, "include")) | 212 | } else if (!strcmp(name, "include")) |
213 | cgit_read_config(value, cgit_global_config_cb); | 213 | cgit_read_config(value, cgit_global_config_cb); |
214 | } | 214 | } |
215 | 215 | ||
216 | void cgit_querystring_cb(const char *name, const char *value) | 216 | void cgit_querystring_cb(const char *name, const char *value) |
217 | { | 217 | { |
218 | if (!strcmp(name,"r")) { | 218 | if (!strcmp(name,"r")) { |
219 | cgit_query_repo = xstrdup(value); | 219 | cgit_query_repo = xstrdup(value); |
220 | cgit_repo = cgit_get_repoinfo(value); | 220 | cgit_repo = cgit_get_repoinfo(value); |
221 | } else if (!strcmp(name, "p")) { | 221 | } else if (!strcmp(name, "p")) { |
222 | cgit_query_page = xstrdup(value); | 222 | cgit_query_page = xstrdup(value); |
223 | cgit_cmd = cgit_get_cmd_index(value); | 223 | cgit_cmd = cgit_get_cmd_index(value); |
224 | } else if (!strcmp(name, "url")) { | 224 | } else if (!strcmp(name, "url")) { |
225 | cgit_parse_url(value); | 225 | cgit_parse_url(value); |
226 | } else if (!strcmp(name, "q")) { | 226 | } else if (!strcmp(name, "q")) { |
227 | cgit_query_search = xstrdup(value); | 227 | cgit_query_search = xstrdup(value); |
228 | } else if (!strcmp(name, "h")) { | 228 | } else if (!strcmp(name, "h")) { |
229 | cgit_query_head = xstrdup(value); | 229 | cgit_query_head = xstrdup(value); |
230 | cgit_query_has_symref = 1; | 230 | cgit_query_has_symref = 1; |
231 | } else if (!strcmp(name, "id")) { | 231 | } else if (!strcmp(name, "id")) { |
232 | cgit_query_sha1 = xstrdup(value); | 232 | cgit_query_sha1 = xstrdup(value); |
233 | cgit_query_has_sha1 = 1; | 233 | cgit_query_has_sha1 = 1; |
234 | } else if (!strcmp(name, "id2")) { | 234 | } else if (!strcmp(name, "id2")) { |
235 | cgit_query_sha2 = xstrdup(value); | 235 | cgit_query_sha2 = xstrdup(value); |
236 | cgit_query_has_sha1 = 1; | 236 | cgit_query_has_sha1 = 1; |
237 | } else if (!strcmp(name, "ofs")) { | 237 | } else if (!strcmp(name, "ofs")) { |
238 | cgit_query_ofs = atoi(value); | 238 | cgit_query_ofs = atoi(value); |
239 | } else if (!strcmp(name, "path")) { | 239 | } else if (!strcmp(name, "path")) { |
240 | cgit_query_path = trim_end(value, '/'); | 240 | cgit_query_path = trim_end(value, '/'); |
241 | } else if (!strcmp(name, "name")) { | 241 | } else if (!strcmp(name, "name")) { |
242 | cgit_query_name = xstrdup(value); | 242 | cgit_query_name = xstrdup(value); |
243 | } | 243 | } |
244 | } | 244 | } |
245 | 245 | ||
246 | void *cgit_free_commitinfo(struct commitinfo *info) | 246 | void *cgit_free_commitinfo(struct commitinfo *info) |
247 | { | 247 | { |
248 | free(info->author); | 248 | free(info->author); |
249 | free(info->author_email); | 249 | free(info->author_email); |
250 | free(info->committer); | 250 | free(info->committer); |
251 | free(info->committer_email); | 251 | free(info->committer_email); |
252 | free(info->subject); | 252 | free(info->subject); |
253 | free(info); | 253 | free(info); |
254 | return NULL; | 254 | return NULL; |
255 | } | 255 | } |
256 | 256 | ||
257 | int hextoint(char c) | 257 | int hextoint(char c) |
258 | { | 258 | { |
259 | if (c >= 'a' && c <= 'f') | 259 | if (c >= 'a' && c <= 'f') |
260 | return 10 + c - 'a'; | 260 | return 10 + c - 'a'; |
261 | else if (c >= 'A' && c <= 'F') | 261 | else if (c >= 'A' && c <= 'F') |
262 | return 10 + c - 'A'; | 262 | return 10 + c - 'A'; |
263 | else if (c >= '0' && c <= '9') | 263 | else if (c >= '0' && c <= '9') |
264 | return c - '0'; | 264 | return c - '0'; |
265 | else | 265 | else |
266 | return -1; | 266 | return -1; |
267 | } | 267 | } |
268 | 268 | ||
269 | char *trim_end(const char *str, char c) | 269 | char *trim_end(const char *str, char c) |
270 | { | 270 | { |
271 | int len; | 271 | int len; |
272 | char *s, *t; | 272 | char *s, *t; |
273 | 273 | ||
274 | if (str == NULL) | 274 | if (str == NULL) |
275 | return NULL; | 275 | return NULL; |
276 | t = (char *)str; | 276 | t = (char *)str; |
277 | len = strlen(t); | 277 | len = strlen(t); |
278 | while(len > 0 && t[len - 1] == c) | 278 | while(len > 0 && t[len - 1] == c) |
279 | len--; | 279 | len--; |
280 | 280 | ||
281 | if (len == 0) | 281 | if (len == 0) |
282 | return NULL; | 282 | return NULL; |
283 | 283 | ||
284 | c = t[len]; | 284 | c = t[len]; |
285 | t[len] = '\0'; | 285 | t[len] = '\0'; |
286 | s = xstrdup(t); | 286 | s = xstrdup(t); |
287 | t[len] = c; | 287 | t[len] = c; |
288 | return s; | 288 | return s; |
289 | } | 289 | } |
290 | 290 | ||
291 | void cgit_diff_tree_cb(struct diff_queue_struct *q, | 291 | void cgit_diff_tree_cb(struct diff_queue_struct *q, |
292 | struct diff_options *options, void *data) | 292 | struct diff_options *options, void *data) |
293 | { | 293 | { |
294 | int i; | 294 | int i; |
295 | 295 | ||
296 | for (i = 0; i < q->nr; i++) { | 296 | for (i = 0; i < q->nr; i++) { |
297 | if (q->queue[i]->status == 'U') | 297 | if (q->queue[i]->status == 'U') |
298 | continue; | 298 | continue; |
299 | ((filepair_fn)data)(q->queue[i]); | 299 | ((filepair_fn)data)(q->queue[i]); |
300 | } | 300 | } |
301 | } | 301 | } |
302 | 302 | ||
303 | static int load_mmfile(mmfile_t *file, const unsigned char *sha1) | 303 | static int load_mmfile(mmfile_t *file, const unsigned char *sha1) |
304 | { | 304 | { |
305 | enum object_type type; | 305 | enum object_type type; |
306 | 306 | ||
307 | if (is_null_sha1(sha1)) { | 307 | if (is_null_sha1(sha1)) { |
308 | file->ptr = (char *)""; | 308 | file->ptr = (char *)""; |
309 | file->size = 0; | 309 | file->size = 0; |
310 | } else { | 310 | } else { |
311 | file->ptr = read_sha1_file(sha1, &type, &file->size); | 311 | file->ptr = read_sha1_file(sha1, &type, &file->size); |
312 | } | 312 | } |
313 | return 1; | 313 | return 1; |
314 | } | 314 | } |
315 | 315 | ||
316 | /* | 316 | /* |
317 | * Receive diff-buffers from xdiff and concatenate them as | 317 | * Receive diff-buffers from xdiff and concatenate them as |
318 | * needed across multiple callbacks. | 318 | * needed across multiple callbacks. |
319 | * | 319 | * |
320 | * This is basically a copy of xdiff-interface.c/xdiff_outf(), | 320 | * This is basically a copy of xdiff-interface.c/xdiff_outf(), |
321 | * ripped from git and modified to use globals instead of | 321 | * ripped from git and modified to use globals instead of |
322 | * a special callback-struct. | 322 | * a special callback-struct. |
323 | */ | 323 | */ |
324 | char *diffbuf = NULL; | 324 | char *diffbuf = NULL; |
325 | int buflen = 0; | 325 | int buflen = 0; |
326 | 326 | ||
327 | int filediff_cb(void *priv, mmbuffer_t *mb, int nbuf) | 327 | int filediff_cb(void *priv, mmbuffer_t *mb, int nbuf) |
328 | { | 328 | { |
329 | int i; | 329 | int i; |
330 | 330 | ||
331 | for (i = 0; i < nbuf; i++) { | 331 | for (i = 0; i < nbuf; i++) { |
332 | if (mb[i].ptr[mb[i].size-1] != '\n') { | 332 | if (mb[i].ptr[mb[i].size-1] != '\n') { |
333 | /* Incomplete line */ | 333 | /* Incomplete line */ |
334 | diffbuf = xrealloc(diffbuf, buflen + mb[i].size); | 334 | diffbuf = xrealloc(diffbuf, buflen + mb[i].size); |
335 | memcpy(diffbuf + buflen, mb[i].ptr, mb[i].size); | 335 | memcpy(diffbuf + buflen, mb[i].ptr, mb[i].size); |
336 | buflen += mb[i].size; | 336 | buflen += mb[i].size; |
337 | continue; | 337 | continue; |
338 | } | 338 | } |
339 | 339 | ||
340 | /* we have a complete line */ | 340 | /* we have a complete line */ |
341 | if (!diffbuf) { | 341 | if (!diffbuf) { |
342 | ((linediff_fn)priv)(mb[i].ptr, mb[i].size); | 342 | ((linediff_fn)priv)(mb[i].ptr, mb[i].size); |
343 | continue; | 343 | continue; |
344 | } | 344 | } |
345 | diffbuf = xrealloc(diffbuf, buflen + mb[i].size); | 345 | diffbuf = xrealloc(diffbuf, buflen + mb[i].size); |
346 | memcpy(diffbuf + buflen, mb[i].ptr, mb[i].size); | 346 | memcpy(diffbuf + buflen, mb[i].ptr, mb[i].size); |
347 | ((linediff_fn)priv)(diffbuf, buflen + mb[i].size); | 347 | ((linediff_fn)priv)(diffbuf, buflen + mb[i].size); |
348 | free(diffbuf); | 348 | free(diffbuf); |
349 | diffbuf = NULL; | 349 | diffbuf = NULL; |
350 | buflen = 0; | 350 | buflen = 0; |
351 | } | 351 | } |
352 | if (diffbuf) { | 352 | if (diffbuf) { |
353 | ((linediff_fn)priv)(diffbuf, buflen); | 353 | ((linediff_fn)priv)(diffbuf, buflen); |
354 | free(diffbuf); | 354 | free(diffbuf); |
355 | diffbuf = NULL; | 355 | diffbuf = NULL; |
356 | buflen = 0; | 356 | buflen = 0; |
357 | } | 357 | } |
358 | return 0; | 358 | return 0; |
359 | } | 359 | } |
360 | 360 | ||
361 | int cgit_diff_files(const unsigned char *old_sha1, | 361 | int cgit_diff_files(const unsigned char *old_sha1, |
362 | const unsigned char *new_sha1, | 362 | const unsigned char *new_sha1, |
363 | linediff_fn fn) | 363 | linediff_fn fn) |
364 | { | 364 | { |
365 | mmfile_t file1, file2; | 365 | mmfile_t file1, file2; |
366 | xpparam_t diff_params; | 366 | xpparam_t diff_params; |
367 | xdemitconf_t emit_params; | 367 | xdemitconf_t emit_params; |
368 | xdemitcb_t emit_cb; | 368 | xdemitcb_t emit_cb; |
369 | 369 | ||
370 | if (!load_mmfile(&file1, old_sha1) || !load_mmfile(&file2, new_sha1)) | 370 | if (!load_mmfile(&file1, old_sha1) || !load_mmfile(&file2, new_sha1)) |
371 | return 1; | 371 | return 1; |
372 | 372 | ||
373 | diff_params.flags = XDF_NEED_MINIMAL; | 373 | diff_params.flags = XDF_NEED_MINIMAL; |
374 | emit_params.ctxlen = 3; | 374 | emit_params.ctxlen = 3; |
375 | emit_params.flags = XDL_EMIT_FUNCNAMES; | 375 | emit_params.flags = XDL_EMIT_FUNCNAMES; |
376 | emit_params.find_func = NULL; | ||
376 | emit_cb.outf = filediff_cb; | 377 | emit_cb.outf = filediff_cb; |
377 | emit_cb.priv = fn; | 378 | emit_cb.priv = fn; |
378 | xdl_diff(&file1, &file2, &diff_params, &emit_params, &emit_cb); | 379 | xdl_diff(&file1, &file2, &diff_params, &emit_params, &emit_cb); |
379 | return 0; | 380 | return 0; |
380 | } | 381 | } |
381 | 382 | ||
382 | void cgit_diff_tree(const unsigned char *old_sha1, | 383 | void cgit_diff_tree(const unsigned char *old_sha1, |
383 | const unsigned char *new_sha1, | 384 | const unsigned char *new_sha1, |
384 | filepair_fn fn) | 385 | filepair_fn fn) |
385 | { | 386 | { |
386 | struct diff_options opt; | 387 | struct diff_options opt; |
387 | int ret; | 388 | int ret; |
388 | 389 | ||
389 | diff_setup(&opt); | 390 | diff_setup(&opt); |
390 | opt.output_format = DIFF_FORMAT_CALLBACK; | 391 | opt.output_format = DIFF_FORMAT_CALLBACK; |
391 | opt.detect_rename = 1; | 392 | opt.detect_rename = 1; |
392 | opt.recursive = 1; | 393 | opt.recursive = 1; |
393 | opt.format_callback = cgit_diff_tree_cb; | 394 | opt.format_callback = cgit_diff_tree_cb; |
394 | opt.format_callback_data = fn; | 395 | opt.format_callback_data = fn; |
395 | diff_setup_done(&opt); | 396 | diff_setup_done(&opt); |
396 | 397 | ||
397 | if (old_sha1 && !is_null_sha1(old_sha1)) | 398 | if (old_sha1 && !is_null_sha1(old_sha1)) |
398 | ret = diff_tree_sha1(old_sha1, new_sha1, "", &opt); | 399 | ret = diff_tree_sha1(old_sha1, new_sha1, "", &opt); |
399 | else | 400 | else |
400 | ret = diff_root_tree_sha1(new_sha1, "", &opt); | 401 | ret = diff_root_tree_sha1(new_sha1, "", &opt); |
401 | diffcore_std(&opt); | 402 | diffcore_std(&opt); |
402 | diff_flush(&opt); | 403 | diff_flush(&opt); |
403 | } | 404 | } |
404 | 405 | ||
405 | void cgit_diff_commit(struct commit *commit, filepair_fn fn) | 406 | void cgit_diff_commit(struct commit *commit, filepair_fn fn) |
406 | { | 407 | { |
407 | unsigned char *old_sha1 = NULL; | 408 | unsigned char *old_sha1 = NULL; |
408 | 409 | ||
409 | if (commit->parents) | 410 | if (commit->parents) |
410 | old_sha1 = commit->parents->item->object.sha1; | 411 | old_sha1 = commit->parents->item->object.sha1; |
411 | cgit_diff_tree(old_sha1, commit->object.sha1, fn); | 412 | cgit_diff_tree(old_sha1, commit->object.sha1, fn); |
412 | } | 413 | } |