Dan McGee schrieb: > -static unsigned int hash_obj(struct object *obj, unsigned int n) > +static unsigned int hash_chars(const unsigned char *c, unsigned int n) hash_chars suggests that this function hashes arbitrary character sequences, but it doesn't do that. Wouldn't hash_object_id be a better name? (And the parameter would then obviously be named sha1 or id.) -- Hannes -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html