Skip to content

indexer: remove the stream infix#1918

Merged
vmg merged 1 commit intodevelopmentfrom
cmn/indexer-naming
Nov 1, 2013
Merged

indexer: remove the stream infix#1918
vmg merged 1 commit intodevelopmentfrom
cmn/indexer-naming

Conversation

@carlosmn
Copy link
Member

It was there to keep it apart from the one which read in from a file on
disk. This other indexer does not exist anymore, so there is no need for
anything other than git_indexer to refer to it.

While here, rename _add() function to _append() and _finalize() to
_commit(). The former change is cosmetic, while the latter avoids
talking about "finalizing", which OO languages use to mean something
completely different.

It was there to keep it apart from the one which read in from a file on
disk. This other indexer does not exist anymore, so there is no need for
anything other than git_indexer to refer to it.

While here, rename _add() function to _append() and _finalize() to
_commit(). The former change is cosmetic, while the latter avoids
talking about "finalizing", which OO languages use to mean something
completely different.
@vmg
Copy link
Member

vmg commented Oct 30, 2013

Running test suite again. Some clone tests failed because Bitbucket was down (hahahahahaha).

vmg pushed a commit that referenced this pull request Nov 1, 2013
indexer: remove the stream infix
@vmg vmg merged commit ac5e507 into development Nov 1, 2013
@carlosmn carlosmn deleted the cmn/indexer-naming branch November 5, 2013 21:26
@ben ben mentioned this pull request Feb 24, 2014
34 tasks
phatblat pushed a commit to phatblat/libgit2 that referenced this pull request Sep 13, 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.

2 participants