From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 306 invoked by alias); 9 May 2013 21:21:08 -0000 Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org Received: (qmail 32765 invoked by uid 89); 9 May 2013 21:21:08 -0000 X-Spam-SWARE-Status: No, score=-7.5 required=5.0 tests=AWL,BAYES_00,KHOP_THREADED,RCVD_IN_HOSTKARMA_W,RCVD_IN_HOSTKARMA_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.1 Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.84/v0.84-167-ge50287c) with ESMTP; Thu, 09 May 2013 21:21:03 +0000 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id r49LL1ES026390 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 9 May 2013 17:21:02 -0400 Received: from localhost.localdomain (ovpn-112-16.ams2.redhat.com [10.36.112.16]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id r49LL0rL025665; Thu, 9 May 2013 17:21:00 -0400 Message-ID: <518C133B.6060401@redhat.com> Date: Thu, 09 May 2013 21:21:00 -0000 From: Phil Muldoon MIME-Version: 1.0 To: Tom Tromey CC: gdb-patches@sourceware.org Subject: Re: [PATCH 00/40] add cleanup checker and fix cleanup bugs References: In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-SW-Source: 2013-05/txt/msg00363.txt.bz2 On 09/05/13 19:47, Tom Tromey wrote: > This series adds a cleanup checker and then fixes most of the bugs it > notices. > > The series as a whole was built and regression-tested on x86-64 Fedora > 18. However, there are a couple of patches that I could not test; > e.g., the machoread.c patch. I would appreciate it if someone could > try these on their machine. The branch is archer.git tromey/cleanup-checker. > > In most cases the patches are pretty obvious. I kept them short for > ease of reading. > > The cleanup checker itself is described in patch #1. This patch also > describes some of the > > After this series, there are still some cleanup-related bugs noticed > by the checker. One of these (in dwarf2read.c) is a false report -- > the checker is not perfect and there was no nice workaround here. > > > The remaining errors are: Thanks for doing this, I think it is awesome. I am not going to comment on the other thirty nine patches as the fixes are largely mechanical in nature. My questions refer to future cleanup issues. As evidenced by the mail on gdb@sourceware.org on clean-up issues, I am more interested in how your contribution can work toward future cleanup errors. Will your cleanup checker run as part of the normal build? (or testsuite)? (or ARI)? Or will it be a manual thing? If it is manual, can it be made automatic? I really like the idea of automatic approaches for catching errors like these. If possible I think it should be part of the everyday build process (assuming one has Python installed). Given the number of errors this tool identified, I think these are common logic errors? What, if any, are the limitations of this, and other, GCC plugins to fixing common GDB patterns of errors in a common build scenario? Cheers, Phil