New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Factor 40 and 41 constants from source. #2567

Merged
merged 1 commit into from Sep 17, 2014

Conversation

Projects
None yet
2 participants
@cirosantilli
Contributor

cirosantilli commented Sep 16, 2014

No description provided.

git_oid_fmt(str, &oid);
str[40] = 0;
str[GIT_OID_HEXSZ+1] = 0;

This comment has been minimized.

@vmg

vmg Sep 16, 2014

Member

Overflow :)

This comment has been minimized.

@cirosantilli

cirosantilli Sep 16, 2014

Contributor

It's inside a comment, so it doesn't count ;)

@cirosantilli cirosantilli force-pushed the cirosantilli:factor-41 branch from cf14daf to 3b2cb2c Sep 16, 2014

vmg added a commit that referenced this pull request Sep 17, 2014

Merge pull request #2567 from cirosantilli/factor-41
Factor 40 and 41 constants from source.

@vmg vmg merged commit 14556cb into libgit2:master Sep 17, 2014

1 check passed

continuous-integration/travis-ci The Travis CI build passed
Details
@vmg

This comment has been minimized.

Member

vmg commented Sep 17, 2014

Thanks! :)

@cirosantilli cirosantilli deleted the cirosantilli:factor-41 branch Sep 17, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment