From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 3455 invoked by alias); 15 Apr 2011 08:20:13 -0000 Received: (qmail 3427 invoked by uid 22791); 15 Apr 2011 08:20:11 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,RFC_ABUSE_POST,TW_BG X-Spam-Check-By: sourceware.org Received: from mail-pw0-f47.google.com (HELO mail-pw0-f47.google.com) (209.85.160.47) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Fri, 15 Apr 2011 08:20:06 +0000 Received: by pwj9 with SMTP id 9so1181878pwj.20 for ; Fri, 15 Apr 2011 01:20:06 -0700 (PDT) MIME-Version: 1.0 Received: by 10.68.6.66 with SMTP id y2mr605215pby.138.1302855605322; Fri, 15 Apr 2011 01:20:05 -0700 (PDT) Received: by 10.68.42.34 with HTTP; Fri, 15 Apr 2011 01:20:05 -0700 (PDT) In-Reply-To: <87fwpkq79r.fsf@rho.meyering.net> References: <87zkp9zmq0.fsf@rho.meyering.net> <877hc9r8w6.fsf_-_@rho.meyering.net> <877hc9pkhp.fsf_-_@rho.meyering.net> <87ei5w8nt7.fsf@rho.meyering.net> <87fwpkq79r.fsf@rho.meyering.net> Date: Fri, 15 Apr 2011 08:48:00 -0000 Message-ID: Subject: Re: [PATCH v3] Re: avoid useless if-before-free tests From: Janne Blomqvist To: Jim Meyering Cc: "Joseph S. Myers" , gcc-patches@gcc.gnu.org, java-patches@gcc.gnu.org, fortran@gcc.gnu.org, gcc mailing list Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable 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: 2011-04/txt/msg01153.txt.bz2 On Fri, Apr 15, 2011 at 10:54, Jim Meyering wrote: > Janne Blomqvist wrote: > >> On Thu, Mar 24, 2011 at 18:51, Jim Meyering wrote: >>> Janne Blomqvist wrote: >>>> On Tue, Mar 8, 2011 at 19:53, Jim Meyering wrote: >>>>> Relative to v2, I've added libgo/ to the list of exempt directories a= nd added >>>>> this recently discussed gfc_free patch, at the request of Tobias Burn= us. >>>>> Also, I corrected an error in fortran's ChangeLog and removed all >>>>> whitespace changes from all ChangeLog files. >>>> >>>> The libgfortran changes are Ok for 4.7. >>>> >>>> For the gfortran frontend (gcc/fortran/*) I'd prefer if you'd >>>> >>>> - Replace all calls to "gfc_free (x)" with "free (x)". >>>> - Remove the gfc_free() function and prototype. >>>> - Remove the free() macro which currently prevents calling free() dire= ctly. >>> >>> Following up, I've refreshed the series but hit a minor snag >>> while converting new uses of gfc_free, removing new tests-before-free >>> and merging/reordering changes. >>> >>> Applying this fix first makes my problem go away: >> >> [snip] >> >> So, what's the plan here? Do you plan to get a GCC account, do you >> already have one, or what? Now that 4.7 is open for development, it's >> perhaps the right time to poke the maintainers to get this patch in. >> >> If you don't have a GCC account, as one of the Fortran maintainers I >> can commit the Fortran and libgfortran parts, but someone else will >> have to do the rest (were they ever approved, BTW?) as I have only >> write after approval privileges for the rest of GCC. > > Can someone add me to the gcc group? =C2=A0That would help. > I already have ssh access to sourceware.org. According to http://gcc.gnu.org/svnwrite.html , you should send an email to overseers(at)gcc.gnu.org . It says that you need a sponsor and "The steering committee or a well-established GCC maintainer (including reviewers) can approve for write access any person with GNU copyright assignment papers in place and known to submit good patches.". I'm not sure if I'm considered to be well-established enough, so could someone help Jim out here, please? Or, if nobody pipes up within a few days, just mention my name as your sponsor and we'll see if the overseers consider me well-established or not.. ;) --=20 Janne Blomqvist