On Wed, 5 Aug 2009, Junio C Hamano wrote: > > Like this? No, combine it with the other shifts: Yes: > - ctx->size += (unsigned long long) len << 3; > + ctx->size += (unsigned long long) len; No: > + ctx->size <<= 3; /* bytes to bits */ > padlen[0] = htonl(ctx->size >> 32); > padlen[1] = htonl(ctx->size); Do padlen[0] = htonl(ctx->size >> 29); padlen[1] = htonl(ctx->size << 3); instead. Or whatever. Linus -- 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