From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26766 invoked by alias); 6 Jan 2007 19:10:02 -0000 Received: (qmail 26743 invoked by uid 22791); 6 Jan 2007 19:10:01 -0000 X-Spam-Check-By: sourceware.org Received: from ug-out-1314.google.com (HELO ug-out-1314.google.com) (66.249.92.172) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sat, 06 Jan 2007 19:09:55 +0000 Received: by ug-out-1314.google.com with SMTP id s2so5865494uge for ; Sat, 06 Jan 2007 11:09:52 -0800 (PST) Received: by 10.67.101.10 with SMTP id d10mr31396017ugm.1168110592621; Sat, 06 Jan 2007 11:09:52 -0800 (PST) Received: from e177197222.adsl.alicedsl.de ( [85.177.197.222]) by mx.google.com with ESMTP id j1sm37221765ugf.2007.01.06.11.09.52; Sat, 06 Jan 2007 11:09:52 -0800 (PST) From: Daniel Franke To: bkorb@gnu.org Subject: Re: [fixincludes, patch] PR30008 - fixes are not applied to files in non-standard locations (was: fixinclude fnmatch patch) Date: Sat, 06 Jan 2007 19:10:00 -0000 User-Agent: KMail/1.9.5 Cc: gcc-patches@gcc.gnu.org References: <200701061544.14147.franke.daniel@gmail.com> <459FE651.8080008@gnu.org> In-Reply-To: <459FE651.8080008@gnu.org> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-6" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200701062009.43985.franke.daniel@gmail.com> Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org X-SW-Source: 2007-01/txt/msg00483.txt.bz2 On Saturday 06 January 2007 19:11, Bruce Korb wrote: > The attached patch is being applied. Thank you, Daniel. Regards, Bruce I should have been more verbose about that particular minor fix ... Your patch removes these lines: - p_fixd->fd_flags |= FD_SKIP_TEST; - return BOOL_FALSE; in machine_matches(). Without them, I observed that no fixes were applied at all. Thus, my patch added + if (have_match == BOOL_FALSE) + p_fixd->fd_flags |= FD_SKIP_TEST; directly after the for-loop to skip any test where "mach" does not match. Could you please verify?! Thanks Daniel