Skip to content

repository: fix garbage return value#5084

Merged
pks-t merged 1 commit intolibgit2:masterfrom
eaigner:garbage-value
May 24, 2019
Merged

repository: fix garbage return value#5084
pks-t merged 1 commit intolibgit2:masterfrom
eaigner:garbage-value

Conversation

@eaigner
Copy link
Copy Markdown

@eaigner eaigner commented May 23, 2019

error was never initialized and a garbage value returned on success.

error was never initialized and a garbage value returned on success.
@pks-t
Copy link
Copy Markdown
Member

pks-t commented May 24, 2019

Yup. Unlikely to happen as the user would need to set both GIT_REPOSITORY_FOREACH_HEAD_SKIP_REPO and GIT_REPOSITORY_FOREACH_HEAD_SKIP_WORKTREES, but a bug nevertheless. Thanks for catching and fixing!

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