diff options
author | Vicent Marti <tanoku@gmail.com> | 2010-08-08 14:12:17 +0200 |
---|---|---|
committer | Vicent Marti <tanoku@gmail.com> | 2010-08-12 18:48:55 +0200 |
commit | 3315782cb4f2b683c66a53c93aa81de501c5a4ab (patch) | |
tree | e3fa8b56dc17b749cc53520eb350cbcc46101007 /src/commit.h | |
parent | f8758044876b30b0f6482d1fe8c3b1de743f4186 (diff) | |
download | libgit2-3315782cb4f2b683c66a53c93aa81de501c5a4ab.tar.gz |
Redesigned the walking/object lookup interface
The old 'git_revpool' object has been removed and
split into two distinct objects with separate
functionality, in order to have separate methods for
object management and object walking.
* A new object 'git_repository' does the high-level
management of a repository's objects (commits, trees,
tags, etc) on top of a 'git_odb'.
Eventually, it will also manage other repository
attributes (e.g. tag resolution, references, etc).
See: src/git/repository.h
* A new external method
'git_repository_lookup(repo, oid, type)'
has been added to the 'git_repository' API.
All object lookups (git_XXX_lookup()) are now
wrappers to this method, and duplicated code
has been removed. The method does automatic type
checking and returns a generic 'git_revpool_object'
that can be cast to any specific object.
See: src/git/repository.h
* The external methods for object parsing of repository
objects (git_XXX_parse()) have been removed.
Loading objects from the repository is now managed
through the 'lookup' functions. These objects are
loaded with minimal information, and the relevant
parsing is done automatically when the user requests
any of the parsed attributes through accessor methods.
An attribute has been added to 'git_repository' in
order to force the parsing of all the repository objects
immediately after lookup.
See: src/git/commit.h
See: src/git/tag.h
See: src/git/tree.h
* The previous walking functionality of the revpool
is now found in 'git_revwalk', which does the actual
revision walking on a repository; the attributes
when walking through commits in a database have been
decoupled from the actual commit objects.
This increases performance when accessing commits
during the walk and allows to have several
'git_revwalk' instances working at the same time on
top of the same repository, without having to load
commits in memory several times.
See: src/git/revwalk.h
* The old 'git_revpool_table' has been renamed to
'git_hashtable' and now works as a generic hashtable
with support for any kind of object and custom hash
functions.
See: src/hashtable.h
* All the relevant unit tests have been updated, renamed
and grouped accordingly.
Signed-off-by: Vicent Marti <tanoku@gmail.com>
Diffstat (limited to 'src/commit.h')
-rw-r--r-- | src/commit.h | 44 |
1 files changed, 9 insertions, 35 deletions
diff --git a/src/commit.h b/src/commit.h index 028c83712..60e1d1115 100644 --- a/src/commit.h +++ b/src/commit.h @@ -3,26 +3,10 @@ #include "git/commit.h" #include "tree.h" -#include "revobject.h" +#include "repository.h" #include <time.h> -struct git_commit_node { - struct git_commit *commit; - - struct git_commit_node *next; - struct git_commit_node *prev; -}; - -struct git_commit_list { - struct git_commit_node *head; - struct git_commit_node *tail; - size_t size; -}; - -typedef struct git_commit_list git_commit_list; -typedef struct git_commit_node git_commit_node; - #define GIT_COMMIT_TREE (1 << 1) #define GIT_COMMIT_PARENTS (1 << 2) #define GIT_COMMIT_AUTHOR (1 << 3) @@ -32,12 +16,17 @@ typedef struct git_commit_node git_commit_node; #define GIT_COMMIT_MESSAGE_SHORT (1 << 7) #define GIT_COMMIT_FOOTERS (1 << 8) +typedef struct git_commit_parents { + git_commit *commit; + struct git_commit_parents *next; +} git_commit_parents; + struct git_commit { - git_revpool_object object; + git_repository_object object; git_obj odb_object; time_t commit_time; - git_commit_list parents; + git_commit_parents *parents; git_tree *tree; git_person *author; @@ -46,13 +35,8 @@ struct git_commit { char *message; char *message_short; - unsigned short in_degree; unsigned basic_parse:1, - odb_open:1, - seen:1, - uninteresting:1, - topo_delay:1, - flags:25; + odb_open:1; }; void git_commit__free(git_commit *c); @@ -64,15 +48,5 @@ void git_commit__mark_uninteresting(git_commit *commit); int git__parse_oid(git_oid *oid, char **buffer_out, const char *buffer_end, const char *header); int git__parse_person(git_person *person, char **buffer_out, const char *buffer_end, const char *header); -int git_commit_list_push_back(git_commit_list *list, git_commit *commit); -int git_commit_list_push_front(git_commit_list *list, git_commit *commit); - -git_commit *git_commit_list_pop_back(git_commit_list *list); -git_commit *git_commit_list_pop_front(git_commit_list *list); - -void git_commit_list_clear(git_commit_list *list, int free_commits); - -void git_commit_list_timesort(git_commit_list *list); -void git_commit_list_toposort(git_commit_list *list); #endif |