On Fri, 10 Aug 2007, Junio C Hamano wrote:Please don't apply that patch without this trivial fix. I don't know why I didn't notice. It passed all the tests, but it really shouldn't have, and the compiler warned. Linus --- diff --git a/builtin-update-index.c b/builtin-update-index.c index 8d22dfa..a7a4574 100644 --- a/builtin-update-index.c +++ b/builtin-update-index.c @@ -91,7 +91,7 @@ static int add_one_path(struct cache_entry *old, const char *path, int len, stru /* Was the old index entry already up-to-date? */ if (old && !ce_stage(old) && !ce_match_stat(old, st, 0)) - return; + return 0; size = cache_entry_size(len); ce = xcalloc(1, size); - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
| Greg Kroah-Hartman | [PATCH 002/196] Chinese: rephrase English introduction in HOWTO |
| Tarkan Erimer | Re: Dual-Licensing Linux Kernel with GPL V2 and GPL V3 |
| Andrew Morton | Re: -mm merge plans for 2.6.23 -- sys_fallocate |
| Greg KH | Re: [AppArmor 39/45] AppArmor: Profile loading and manipulation, pathname matching |
git: | |
| Gerrit Renker | [PATCH 03/37] dccp: List management for new feature negotiation |
| Arjan van de Ven | Re: [GIT]: Networking |
| Jarek Poplawski | Re: [PATCH] pkt_sched: Destroy gen estimators under rtnl_lock(). |
| Jarek Poplawski | Re: [BUG] New Kernel Bugs |
