From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3661 invoked by alias); 10 Dec 2002 16:13:23 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 3654 invoked from network); 10 Dec 2002 16:13:22 -0000 Received: from unknown (HELO localhost.redhat.com) (216.138.202.10) by sources.redhat.com with SMTP; 10 Dec 2002 16:13:22 -0000 Received: from redhat.com (localhost [127.0.0.1]) by localhost.redhat.com (Postfix) with ESMTP id 0364C3CFE; Tue, 10 Dec 2002 11:13:16 -0500 (EST) Message-ID: <3DF6129C.9000700@redhat.com> Date: Tue, 10 Dec 2002 08:32:00 -0000 From: Andrew Cagney User-Agent: Mozilla/5.0 (X11; U; NetBSD macppc; en-US; rv:1.0.0) Gecko/20020824 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Kevin Buettner Cc: gdb-patches@sources.redhat.com Subject: Re: [RFA] dwarf2read.c: complain() -> complaint() References: <1021209233902.ZM5908@localhost.localdomain> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2002-12/txt/msg00324.txt.bz2 > In a recently submitted patch, I added a new complaint to dwarf2read.c > using the old (deprecated) interface. Andrew asked me to use the new > interface. I've generated a patch for dwarf2read.c which converts all > calls to complain() to complaint(). > > I've written a script to do this transformation. It finds all of the > deprecated_complaint structs and counts the uses of these structs. > (Surprisingly, some were unused!) For cases where there's more than > one occurrence, it creates a new function as suggested in > complaints.h. For the rest, it performs the obvious transformation. > > The script also uses GNU indent to perform localized reindentations of > the affected text. > > If this patch is accepted (and this approach is deemed acceptable), I'll > generate patches for the other files which still use complain(). Definitly fine with the theory. Two reservations (really just one). - how is the result with -Wformat? The reason behind switching from complain() to complaint was to get the parameters checked and hence find some nasty address printing bugs. - I suspect it will need a visual audit to check for any cases of: "%08lx", (long) core_addr_variable Looking through this specific patch, though, turned up no cases where this occured. So you must have that those problems covered. Symtab maintainers? Andrew (PS, yes, I'd noticed that some of the complaints were unused).