On Sat, Mar 07, 2009 at 09:02:10PM +0100, Benjamin Kramer wrote: > these variables were unused and can be removed safely > > builtin-clone: removed unused variable 'option_no_hardlinks' I did a double-take here, as I was just looking at option_no_hardlinks code the other day. But looking at your patch, I think you meant "use_local_hardlinks". > @@ -388,9 +386,6 @@ int cmd_clone(int argc, const char **argv, const > char *prefix) Your patch is damaged due to wrapping here (and several other places). Please check your mailer settings. Other than that, I think it is a fine cleanup. -Peff -- 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
