From: Jeff King Date: Wed, 26 Jul 2017 13:17:29 GMT Subject: Re: [PATCH] fsck: cleanup unused variable Message-ID: <20170726131729.o6bp2qmlqpl674zt@sigill.intra.peff.net> In-Reply-To: <20170726013456.4187-1-jonathantanmy@google.com> On Tue, Jul 25, 2017 at 06:34:56PM -0700, Jonathan Tan wrote: > Remove the unused variable "heads" from cmd_fsck(). > > This variable was made unused in commit c3271a0 ("fsck: do not fallback > "git fsck " to "git fsck"", 2017-01-17). Thanks, I should have caught this then. The patch is obviously correct, though I'm curious why gcc's -Wunused-but-set-variable didn't catch this. -Peff