On Sat, Nov 29, 2008 at 10:00 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Junio C Hamano <gitster@xxxxxxxxx> writes: > >> ... The change is not >> about fixing (your proposed commit log message read "gitweb:fixes to >> gitweb feature check code") as nothing is broken. It is purely about >> futureproofing and I think we should mark it as such. > > Actually, there is a handful clueless/careless callers. Before applying > any of the check => get patch, let's do this as a fix. And this is precisely the reason why the first time I sent the patch I did the restyling in the same go: by not touching the clueless/careless callers and instead bringing gitweb_check_feature to act in scalar context, it would automatically fix those broken usages, and it made sense to convert all array-to-get-a-scalar assignmets as well. -- Giuseppe "Oblomov" Bilotta -- 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
