Skip to content

Unresolved getters for git_merge_driver_source struct were added - #4153

Closed
gumenikav wants to merge 2 commits into
libgit2:masterfrom
gumenikav:merge_driver_source_info
Closed

Unresolved getters for git_merge_driver_source struct were added#4153
gumenikav wants to merge 2 commits into
libgit2:masterfrom
gumenikav:merge_driver_source_info

Conversation

@gumenikav

Copy link
Copy Markdown

Unresolved getters for git_merge_driver_source struct were added:

  • git_repository* git_merge_driver_source_repo(const git_merge_driver_source *src)
  • git_index_entry* git_merge_driver_source_ancestor(const git_merge_driver_source *src)
  • git_index_entry* git_merge_driver_source_ours(const git_merge_driver_source *src)
  • git_index_entry* git_merge_driver_source_theirs(const git_merge_driver_source *src)
  • git_merge_file_options* git_merge_driver_source_file_options(const git_merge_driver_source *src)

@pks-t pks-t left a comment

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.

Thanks for this, the implementation is obviously correct. Could you please just remove the single docstring in merge_driver.c?

Comment thread src/merge_driver.c Outdated

static void git_merge_driver_global_shutdown(void);

/** Get the repository that the source data is coming from. */

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.

This docstring is already part of merge_driver.h.

@gumenikav

Copy link
Copy Markdown
Author

No problem. I removed docstring.

@pks-t

pks-t commented Mar 15, 2017

Copy link
Copy Markdown
Member

I've squashed your two commits together and manually merged it via 8051b47.

Thanks for your work!

@pks-t pks-t closed this Mar 15, 2017
@gumenikav
gumenikav deleted the merge_driver_source_info branch March 15, 2017 09:13
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.

2 participants