mirror of
https://github.com/git/git.git
synced 2026-01-12 05:43:12 +09:00
9e20442298
Finish the change started in the preceding commit and allow an early
return from "xdiff_emit_line_fn" callbacks, this will allows
diffcore-pickaxe.c to save itself redundant work.
Our xdiff interface also had the limitation of not being able to abort
early since the beginning, see d9ea73e0564 (combine-diff: refactor
built-in xdiff interface., 2006-04-05). Although at that time
"xdiff_emit_line_fn" was called "xdiff_emit_consume_fn", and
"xdiff_emit_hunk_fn" didn't exist yet.
There was some work in this area of xdiff-interface.[ch] recently with
3b40a090fd4 (diff: avoid generating unused hunk header lines,
2018-11-02) and 7c61e25fbf1 (diff: use hunk callback for word-diff,
2018-11-02).
In combination those two changes allow us to not do any work on the
hunks and diff at all, but didn't change the status quo with regards
to consumers that e.g. want the diff lines, but might want to abort
early.
Whereas now we can abort e.g. on the first "-line" of a 1000 line diff
if that's all we needed.
This interface is rather scary as noted in the comment to
xdiff-interface.h being added here, as noted there a future change
could add more exit codes, and hack xdl_emit_diff() and friends to
ignore or skip things more selectively as a result.
I did not see an inherent reason for why xdl_emit_{diffrec,record}()
could not be changed to ferry the "xdiff_emit_line_fn" error code
upwards instead of returning -1 on all "ret < 0".
But doing so would require corresponding changes in xdl_emit_diff(),
xdl_diff(). I didn't see any issue with narrowly doing that to
accomplish what I needed here, but it would leave xdiff's own return
values in an inconsistent state.
Instead I've left it at returning a more conventional (for git's own
codebase) 1 for an early return, and translating it (or rather, all
non-zero) to -1 for xdiff's consumption.
The reason for most of the "stop" complexity in xdiff_outf() is
because we want to be able to abort early, but do so in a way that
doesn't skip the appropriate strbuf_reset() invocations.
Signed-off-by: Ævar Arnfjörð Bjarmason <avarab@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
77 lines
2.8 KiB
C
77 lines
2.8 KiB
C
#ifndef XDIFF_INTERFACE_H
|
|
#define XDIFF_INTERFACE_H
|
|
|
|
#include "cache.h"
|
|
#include "xdiff/xdiff.h"
|
|
|
|
/*
|
|
* xdiff isn't equipped to handle content over a gigabyte;
|
|
* we make the cutoff 1GB - 1MB to give some breathing
|
|
* room for constant-sized additions (e.g., merge markers)
|
|
*/
|
|
#define MAX_XDIFF_SIZE (1024UL * 1024 * 1023)
|
|
|
|
/**
|
|
* The `xdiff_emit_line_fn` function can return 1 to abort early, or 0
|
|
* to continue processing. Note that doing so is an all-or-nothing
|
|
* affair, as returning 1 will return all the way to the top-level,
|
|
* e.g. the xdi_diff_outf() call to generate the diff.
|
|
*
|
|
* Thus returning 1 means you won't be getting any more diff lines. If
|
|
* you need something in-between those two options you'll to use
|
|
* `xdl_emit_hunk_consume_func_t` and implement your own version of
|
|
* xdl_emit_diff().
|
|
*
|
|
* We may extend the interface in the future to understand other more
|
|
* granular return values. While you should return 1 to exit early,
|
|
* doing so will currently make your early return indistinguishable
|
|
* from an error internal to xdiff, xdiff itself will see that
|
|
* non-zero return and translate it to -1.
|
|
*/
|
|
typedef int (*xdiff_emit_line_fn)(void *, char *, unsigned long);
|
|
typedef void (*xdiff_emit_hunk_fn)(void *data,
|
|
long old_begin, long old_nr,
|
|
long new_begin, long new_nr,
|
|
const char *func, long funclen);
|
|
|
|
int xdi_diff(mmfile_t *mf1, mmfile_t *mf2, xpparam_t const *xpp, xdemitconf_t const *xecfg, xdemitcb_t *ecb);
|
|
int xdi_diff_outf(mmfile_t *mf1, mmfile_t *mf2,
|
|
xdiff_emit_hunk_fn hunk_fn,
|
|
xdiff_emit_line_fn line_fn,
|
|
void *consume_callback_data,
|
|
xpparam_t const *xpp, xdemitconf_t const *xecfg);
|
|
int read_mmfile(mmfile_t *ptr, const char *filename);
|
|
void read_mmblob(mmfile_t *ptr, const struct object_id *oid);
|
|
int buffer_is_binary(const char *ptr, unsigned long size);
|
|
|
|
void xdiff_set_find_func(xdemitconf_t *xecfg, const char *line, int cflags);
|
|
void xdiff_clear_find_func(xdemitconf_t *xecfg);
|
|
int git_xmerge_config(const char *var, const char *value, void *cb);
|
|
extern int git_xmerge_style;
|
|
|
|
/*
|
|
* Can be used as a no-op hunk_fn for xdi_diff_outf(), since a NULL
|
|
* one just sends the hunk line to the line_fn callback).
|
|
*/
|
|
void discard_hunk_line(void *priv,
|
|
long ob, long on, long nb, long nn,
|
|
const char *func, long funclen);
|
|
|
|
/*
|
|
* Compare the strings l1 with l2 which are of size s1 and s2 respectively.
|
|
* Returns 1 if the strings are deemed equal, 0 otherwise.
|
|
* The `flags` given as XDF_WHITESPACE_FLAGS determine how white spaces
|
|
* are treated for the comparison.
|
|
*/
|
|
int xdiff_compare_lines(const char *l1, long s1,
|
|
const char *l2, long s2, long flags);
|
|
|
|
/*
|
|
* Returns a hash of the string s of length len.
|
|
* The `flags` given as XDF_WHITESPACE_FLAGS determine how white spaces
|
|
* are treated for the hash.
|
|
*/
|
|
unsigned long xdiff_hash_string(const char *s, size_t len, long flags);
|
|
|
|
#endif
|