diff options
author | Russell Belfer <rb@github.com> | 2012-06-08 22:56:24 +0400 |
---|---|---|
committer | Russell Belfer <rb@github.com> | 2012-06-08 23:11:13 +0400 |
commit | 145e696b498a046762e4df9045c9b71440308486 (patch) | |
tree | b37682dece97e69e2be1056d656dc8d6fc5d3155 /examples/diff.c | |
parent | 0abd724454078f2089701b54be94df7306dcfb8e (diff) |
Minor fixes, cleanups, and clarifications
There are three actual changes in this commit:
1. When the trailing newline of a file is removed in a diff, the
change will now be reported with `GIT_DIFF_LINE_DEL_EOFNL` passed
to the callback. Previously, the `ADD_EOFNL` constant was given
which was just an error in my understanding of when the various
circumstances arose. `GIT_DIFF_LINE_ADD_EOFNL` is deprecated and
should never be generated. A new newline is simply an `ADD`.
2. Rewrote the `diff_delta__merge_like_cgit` function that contains
the core logic of the `git_diff_merge` implementation. The new
version doesn't actually have significantly different behavior,
but the logic should be much more obvious, I think.
3. Fixed a bug in `git_diff_merge` where it freed a string pool
while some of the string data was still in use. This led to
`git_diff_print_patch` accessing memory that had been freed.
The rest of this commit contains improved documentation in `diff.h`
to make the behavior and the equivalencies with core git clearer,
and a bunch of new tests to cover the various cases, oh and a minor
simplification of `examples/diff.c`.
Diffstat (limited to 'examples/diff.c')
-rw-r--r-- | examples/diff.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/examples/diff.c b/examples/diff.c index 1b4ab549b..b72a75e1c 100644 --- a/examples/diff.c +++ b/examples/diff.c @@ -185,9 +185,7 @@ int main(int argc, char *argv[]) /* open repo */ - check(git_repository_discover(path, sizeof(path), dir, 0, "/"), - "Could not discover repository"); - check(git_repository_open(&repo, path), + check(git_repository_open_ext(&repo, dir, 0, NULL), "Could not open repository"); if (treeish1) |