Implemented parameters for Diff.find_similar() by ArnoVanLumig · Pull Request #396 · libgit2/pygit2 · GitHub
Skip to content

Implemented parameters for Diff.find_similar()#396

Merged
jdavid merged 2 commits into
libgit2:masterfrom
ArnoVanLumig:master
Jul 28, 2014
Merged

Implemented parameters for Diff.find_similar()#396
jdavid merged 2 commits into
libgit2:masterfrom
ArnoVanLumig:master

Conversation

@ArnoVanLumig

Copy link
Copy Markdown

Diff.find_similar now supports the parameters that are exposed by libgit2's git_diff_find_similar.

Parameters supported are rename_threshold, copy_threshold,
rename_from_rewrite_threshold, break_rewrite_threshold and rename_limit

Diff.find_similar now supports the parameters that are exposed by
libgit2's git_diff_find_similar.

Parameters supported are rename_threshold, copy_threshold,
rename_from_rewrite_threshold, break_rewrite_threshold and rename_limit
@ArnoVanLumig

Copy link
Copy Markdown
Author

Comment thread src/diff.c Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All of these values will be unitialised if they're not passed. Instead of this, you should let PyArg_ParseTupleAndKeywords write into the options structure directly.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

I actually figured that out myself but forgot to commit and push it... Thanks for the heads-up!

@ArnoVanLumig

Copy link
Copy Markdown
Author

@jdavid jdavid merged commit e31f0ac into libgit2:master Jul 28, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants