From patchwork Sat Sep 25 07:07:16 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tobias Burnus X-Patchwork-Id: 65737 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from sourceware.org (server1.sourceware.org [209.132.180.131]) by ozlabs.org (Postfix) with SMTP id 91F66B70DA for ; Sat, 25 Sep 2010 17:07:47 +1000 (EST) Received: (qmail 28409 invoked by alias); 25 Sep 2010 07:07:28 -0000 Received: (qmail 28367 invoked by uid 22791); 25 Sep 2010 07:07:26 -0000 X-SWARE-Spam-Status: No, hits=-1.8 required=5.0 tests=AWL, BAYES_00, RCVD_IN_DNSWL_NONE X-Spam-Check-By: sourceware.org Received: from mx02.qsc.de (HELO mx02.qsc.de) (213.148.130.14) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sat, 25 Sep 2010 07:07:20 +0000 Received: from [192.168.178.22] (port-92-204-109-183.dynamic.qsc.de [92.204.109.183]) by mx02.qsc.de (Postfix) with ESMTP id E34EC1E922; Sat, 25 Sep 2010 09:07:16 +0200 (CEST) Message-ID: <4C9D9FA4.8040607@net-b.de> Date: Sat, 25 Sep 2010 09:07:16 +0200 From: Tobias Burnus User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; de; rv:1.9.2.9) Gecko/20100914 SUSE/3.1.4 Thunderbird/3.1.4 MIME-Version: 1.0 To: Steve Kargl CC: Jerry DeLisle , fortran@gcc.gnu.org, gcc-patches@gcc.gnu.org Subject: Re: [PATCH, fortran] Enforce F2003 C1202 References: <20100925034244.GA80437@troutmask.apl.washington.edu> <4C9D89AC.8010601@frontier.com> <20100925055623.GA81179@troutmask.apl.washington.edu> In-Reply-To: <20100925055623.GA81179@troutmask.apl.washington.edu> Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Delivered-To: mailing list gcc-patches@gcc.gnu.org Steve Kargl wrote: > On Fri, Sep 24, 2010 at 10:33:32PM -0700, Jerry DeLisle wrote: >> On 09/24/2010 08:42 PM, Steve Kargl wrote: >>> Regression tested on x86_64-*-freebsd. >>> OK for trunk > Committed revision 164616. The committed patch did not compile here as you strip off the "const" from the pointer - given an error (due to -Werror). Thus, I have committed the attached patch as obvious. Rev. 164617, Tobias Index: gcc/fortran/interface.c =================================================================== --- gcc/fortran/interface.c (Revision 164616) +++ gcc/fortran/interface.c (Arbeitskopie) @@ -320,7 +320,7 @@ } else { - char *s1, *s2; + const char *s1, *s2; s1 = gfc_op2string (current_interface.op); s2 = gfc_op2string (op); Index: gcc/fortran/ChangeLog =================================================================== --- gcc/fortran/ChangeLog (Revision 164616) +++ gcc/fortran/ChangeLog (Arbeitskopie) @@ -1,6 +1,11 @@ +2010-09-25 Tobias Burnus + + * interface.c (gfc_match_end_interface): Constify char pointer + to fix warning. + 2010-09-24 Steven G. Kargl < kargl@gcc.gnu.org> - * fortran/interface.c (gfc_match_end_interface): Deal with user defined + * interface.c (gfc_match_end_interface): Deal with user defined operators that overload rational operators and C1202. 2010-09-24 Tobias Burnus