mirror of
https://github.com/git/git.git
synced 2024-11-18 19:13:58 +01:00
[PATCH] Port fsck-cache to use parsing functions
This ports fsck-cache to use parsing functions. Note that performance could be improved here by only reading each object once, but this requires somewhat more complicated flow control. Signed-Off-By: Daniel Barkalow <barkalow@iabervon.org> Signed-off-by: Linus Torvalds <torvalds@osdl.org>
This commit is contained in:
parent
5873b67eef
commit
ff5ebe39b0
4
Makefile
4
Makefile
@ -43,8 +43,8 @@ commit-tree: commit-tree.o read-cache.o
|
||||
cat-file: cat-file.o read-cache.o
|
||||
$(CC) $(CFLAGS) -o cat-file cat-file.o read-cache.o $(LIBS)
|
||||
|
||||
fsck-cache: fsck-cache.o read-cache.o
|
||||
$(CC) $(CFLAGS) -o fsck-cache fsck-cache.o read-cache.o $(LIBS)
|
||||
fsck-cache: fsck-cache.o read-cache.o object.o commit.o tree.o blob.o
|
||||
$(CC) $(CFLAGS) -o fsck-cache fsck-cache.o read-cache.o object.o commit.o tree.o blob.o $(LIBS)
|
||||
|
||||
checkout-cache: checkout-cache.o read-cache.o
|
||||
$(CC) $(CFLAGS) -o checkout-cache checkout-cache.o read-cache.o $(LIBS)
|
||||
|
106
fsck-cache.c
106
fsck-cache.c
@ -3,7 +3,11 @@
|
||||
#include <sys/types.h>
|
||||
#include <dirent.h>
|
||||
|
||||
#include "revision.h"
|
||||
#include "commit.h"
|
||||
#include "tree.h"
|
||||
#include "blob.h"
|
||||
|
||||
#define REACHABLE 0x0001
|
||||
|
||||
static int show_unreachable = 0;
|
||||
static unsigned char head_sha1[20];
|
||||
@ -13,96 +17,54 @@ static void check_connectivity(void)
|
||||
int i;
|
||||
|
||||
/* Look up all the requirements, warn about missing objects.. */
|
||||
for (i = 0; i < nr_revs; i++) {
|
||||
struct revision *rev = revs[i];
|
||||
for (i = 0; i < nr_objs; i++) {
|
||||
struct object *obj = objs[i];
|
||||
|
||||
if (show_unreachable && !(rev->flags & REACHABLE)) {
|
||||
printf("unreachable %s\n", sha1_to_hex(rev->sha1));
|
||||
if (show_unreachable && !(obj->flags & REACHABLE)) {
|
||||
printf("unreachable %s\n", sha1_to_hex(obj->sha1));
|
||||
continue;
|
||||
}
|
||||
|
||||
switch (rev->flags & (SEEN | USED)) {
|
||||
case 0:
|
||||
printf("bad %s\n", sha1_to_hex(rev->sha1));
|
||||
break;
|
||||
case USED:
|
||||
printf("missing %s\n", sha1_to_hex(rev->sha1));
|
||||
break;
|
||||
case SEEN:
|
||||
printf("dangling %s\n", sha1_to_hex(rev->sha1));
|
||||
break;
|
||||
if (!obj->parsed) {
|
||||
printf("missing %s %s\n", obj->type,
|
||||
sha1_to_hex(obj->sha1));
|
||||
}
|
||||
if (!obj->used) {
|
||||
printf("dangling %s %s\n", obj->type,
|
||||
sha1_to_hex(obj->sha1));
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
static void mark_needs_sha1(unsigned char *parent, const char * tag, unsigned char *child)
|
||||
{
|
||||
struct revision * child_rev = add_relationship(lookup_rev(parent), child);
|
||||
child_rev->flags |= USED;
|
||||
}
|
||||
|
||||
static int mark_sha1_seen(unsigned char *sha1, char *tag)
|
||||
{
|
||||
struct revision *rev = lookup_rev(sha1);
|
||||
|
||||
rev->flags |= SEEN;
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int fsck_tree(unsigned char *sha1, void *data, unsigned long size)
|
||||
{
|
||||
int warn_old_tree = 1;
|
||||
|
||||
while (size) {
|
||||
int len = 1+strlen(data);
|
||||
unsigned char *file_sha1 = data + len;
|
||||
char *path = strchr(data, ' ');
|
||||
unsigned int mode;
|
||||
if (size < len + 20 || !path || sscanf(data, "%o", &mode) != 1)
|
||||
return -1;
|
||||
|
||||
/* Warn about trees that don't do the recursive thing.. */
|
||||
if (warn_old_tree && strchr(path, '/')) {
|
||||
fprintf(stderr, "warning: fsck-cache: tree %s has full pathnames in it\n", sha1_to_hex(sha1));
|
||||
warn_old_tree = 0;
|
||||
}
|
||||
|
||||
data += len + 20;
|
||||
size -= len + 20;
|
||||
mark_needs_sha1(sha1, S_ISDIR(mode) ? "tree" : "blob", file_sha1);
|
||||
struct tree *item = lookup_tree(sha1);
|
||||
if (parse_tree(item))
|
||||
return -1;
|
||||
if (item->has_full_path) {
|
||||
fprintf(stderr, "warning: fsck-cache: tree %s "
|
||||
"has full pathnames in it\n", sha1_to_hex(sha1));
|
||||
}
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int fsck_commit(unsigned char *sha1, void *data, unsigned long size)
|
||||
{
|
||||
int parents;
|
||||
unsigned char tree_sha1[20];
|
||||
unsigned char parent_sha1[20];
|
||||
|
||||
if (memcmp(data, "tree ", 5))
|
||||
struct commit *commit = lookup_commit(sha1);
|
||||
if (parse_commit(commit))
|
||||
return -1;
|
||||
if (get_sha1_hex(data + 5, tree_sha1) < 0)
|
||||
if (!commit->tree)
|
||||
return -1;
|
||||
mark_needs_sha1(sha1, "tree", tree_sha1);
|
||||
data += 5 + 40 + 1; /* "tree " + <hex sha1> + '\n' */
|
||||
parents = 0;
|
||||
while (!memcmp(data, "parent ", 7)) {
|
||||
if (get_sha1_hex(data + 7, parent_sha1) < 0)
|
||||
return -1;
|
||||
mark_needs_sha1(sha1, "commit", parent_sha1);
|
||||
data += 7 + 40 + 1; /* "parent " + <hex sha1> + '\n' */
|
||||
parents++;
|
||||
}
|
||||
if (!parents)
|
||||
if (!commit->parents)
|
||||
printf("root %s\n", sha1_to_hex(sha1));
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int fsck_entry(unsigned char *sha1, char *tag, void *data, unsigned long size)
|
||||
static int fsck_entry(unsigned char *sha1, char *tag, void *data,
|
||||
unsigned long size)
|
||||
{
|
||||
if (!strcmp(tag, "blob")) {
|
||||
/* Nothing to check */;
|
||||
lookup_blob(sha1); /* Nothing to check; but notice it. */
|
||||
} else if (!strcmp(tag, "tree")) {
|
||||
if (fsck_tree(sha1, data, size) < 0)
|
||||
return -1;
|
||||
@ -111,7 +73,7 @@ static int fsck_entry(unsigned char *sha1, char *tag, void *data, unsigned long
|
||||
return -1;
|
||||
} else
|
||||
return -1;
|
||||
return mark_sha1_seen(sha1, tag);
|
||||
return 0;
|
||||
}
|
||||
|
||||
static int fsck_name(char *hex)
|
||||
@ -125,7 +87,8 @@ static int fsck_name(char *hex)
|
||||
unsigned long size;
|
||||
void *buffer = NULL;
|
||||
if (!check_sha1_signature(sha1, map, mapsize))
|
||||
buffer = unpack_sha1_file(map, mapsize, type, &size);
|
||||
buffer = unpack_sha1_file(map, mapsize, type,
|
||||
&size);
|
||||
munmap(map, mapsize);
|
||||
if (buffer && !fsck_entry(sha1, type, buffer, size))
|
||||
return 0;
|
||||
@ -186,7 +149,10 @@ int main(int argc, char **argv)
|
||||
continue;
|
||||
}
|
||||
if (!get_sha1_hex(argv[i], head_sha1)) {
|
||||
mark_reachable(lookup_rev(head_sha1), REACHABLE);
|
||||
struct object *obj =
|
||||
&lookup_commit(head_sha1)->object;
|
||||
obj->used = 1;
|
||||
mark_reachable(obj, REACHABLE);
|
||||
heads++;
|
||||
continue;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user