Skip to content

object: deprecate git_object__size for removal#5192

Merged
carlosmn merged 1 commit intomasterfrom
cmn/object-size-nopublic
Jul 29, 2019
Merged

object: deprecate git_object__size for removal#5192
carlosmn merged 1 commit intomasterfrom
cmn/object-size-nopublic

Conversation

@carlosmn
Copy link
Copy Markdown
Member

In #5118 we remove the double-underscore to make it a normally-named public
function. However, this is not an interesting function outside of the library
and it takes up a name for something that could be more useful.

Remove the single-underscore version as we have not done any releases with it.

Copy link
Copy Markdown
Member

@ethomson ethomson left a comment

Choose a reason for hiding this comment

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

🔥

In #5118 we remove the double-underscore to make it a normally-named public
function. However, this is not an interesting function outside of the library
and it takes up a name for something that could be more useful.

Remove the single-underscore version as we have not done any releases with it.
@carlosmn carlosmn force-pushed the cmn/object-size-nopublic branch from 3de2e3f to c8e249b Compare July 29, 2019 09:15
@carlosmn carlosmn merged commit f039c83 into master Jul 29, 2019
@carlosmn carlosmn deleted the cmn/object-size-nopublic branch July 29, 2019 11:02
@carlosmn
Copy link
Copy Markdown
Member Author

If it turns out someone really needs it we can bring it back as git_object_sizeof or something like that. I'm glad to have this name free now for a a getter for the size of the git object.

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