From mboxrd@z Thu Jan 1 00:00:00 1970 From: Junio C Hamano Subject: Re: [PATCH 2/2] read-cache.c: fix timestamp comparison Date: Tue, 22 Jan 2008 01:47:59 -0800 Message-ID: <7v3asqry28.fsf@gitster.siamese.dyndns.org> References: <1200022189-2400-1-git-send-email-mlevedahl@gmail.com> <7vir1xmazm.fsf@gitster.siamese.dyndns.org> <7v63xrh3mw.fsf_-_@gitster.siamese.dyndns.org> <7vfxwvfmd8.fsf_-_@gitster.siamese.dyndns.org> <7vr6gb3nv1.fsf@gitster.siamese.dyndns.org> <7vd4rv3ds5.fsf@gitster.siamese.dyndns.org> <7vtzl71x1c.fsf@gitster.siamese.dyndns.org> <7vprvv1wnu.fsf@gitster.siamese.dyndns.org> <7vlk6j1wjj.fsf@gitster.siamese.dyndns.org> <7vhch71vvb.fsf@gitster.siamese.dyndns.org> <7v8x2j1sul.fsf@gitster.siamese.dyndns.org> <7vzluzzhud.fsf_-_@gitster.siamese.dyndns.org> <7vabmyykvg.fsf@gitster.siamese.dyndns.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: git@vger.kernel.org To: Linus Torvalds X-From: git-owner@vger.kernel.org Tue Jan 22 10:48:58 2008 Return-path: Envelope-to: gcvg-git-2@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by lo.gmane.org with esmtp (Exim 4.50) id 1JHFkO-0003ql-3q for gcvg-git-2@gmane.org; Tue, 22 Jan 2008 10:48:48 +0100 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755728AbYAVJsI (ORCPT ); Tue, 22 Jan 2008 04:48:08 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1756367AbYAVJsH (ORCPT ); Tue, 22 Jan 2008 04:48:07 -0500 Received: from a-sasl-quonix.sasl.smtp.pobox.com ([208.72.237.25]:64681 "EHLO sasl.smtp.pobox.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755671AbYAVJsG (ORCPT ); Tue, 22 Jan 2008 04:48:06 -0500 Received: from a-sasl-quonix (localhost [127.0.0.1]) by a-sasl-quonix.pobox.com (Postfix) with ESMTP id 245B64329; Tue, 22 Jan 2008 04:48:04 -0500 (EST) Received: from pobox.com (ip68-225-240-77.oc.oc.cox.net [68.225.240.77]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by a-sasl-quonix.pobox.com (Postfix) with ESMTP id 98C2B4328; Tue, 22 Jan 2008 04:48:01 -0500 (EST) In-Reply-To: (Linus Torvalds's message of "Mon, 21 Jan 2008 13:22:15 -0800 (PST)") User-Agent: Gnus/5.110006 (No Gnus v0.6) Emacs/21.4 (gnu/linux) Sender: git-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: git@vger.kernel.org Archived-At: Linus Torvalds writes: > The stupid thing is that I literally _grepped_ for the code testing > "ce_mode". > > I had missed that one because I had looked for things like > > if ([!].*ce_mode) > > but that switch statement meant that the comparison to zero was > non-local and my grep didn't see it. Would a patch to sparse help you in cases like this?