From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 21196 invoked by alias); 22 Oct 2011 00:03:22 -0000 Received: (qmail 21182 invoked by uid 22791); 22 Oct 2011 00:03:21 -0000 X-SWARE-Spam-Status: No, hits=-2.3 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW X-Spam-Check-By: sourceware.org Received: from mail-bw0-f47.google.com (HELO mail-bw0-f47.google.com) (209.85.214.47) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Sat, 22 Oct 2011 00:03:06 +0000 Received: by bkat8 with SMTP id t8so6131953bka.20 for ; Fri, 21 Oct 2011 17:03:04 -0700 (PDT) MIME-Version: 1.0 Received: by 10.223.81.205 with SMTP id y13mr27632055fak.34.1319241784427; Fri, 21 Oct 2011 17:03:04 -0700 (PDT) Received: by 10.152.25.98 with HTTP; Fri, 21 Oct 2011 17:03:04 -0700 (PDT) In-Reply-To: <4EA1B14E.80307@oracle.com> References: <4E9F4265.6080809@oracle.com> <4E9F4FEA.6050107@redhat.com> <4E9F5F0C.1050607@oracle.com> <4E9F66CC.3050604@oracle.com> <4E9F736D.8000404@oracle.com> <4EA1B14E.80307@oracle.com> Date: Sat, 22 Oct 2011 05:54:00 -0000 Message-ID: Subject: Re: [C++ Patch] PR 48630 (PR 31423) From: Gabriel Dos Reis To: Paolo Carlini Cc: Jason Merrill , "gcc-patches@gcc.gnu.org" Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes 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-10/txt/msg02048.txt.bz2 On Fri, Oct 21, 2011 at 12:52 PM, Paolo Carlini wrote: > Hi again, >>> >>> Another acceptable fix is to >>> =A0 -- leave the current diagnostic as is if -fms-extensions >>> =A0 -- suggest '()' is member function >>> =A0 -- otherwise suggest '&'. >> >> Thanks for your help Gaby, in particular about the MS extension which I >> had overlooked completely (as any hard-code Linux guy would ;). Anyway, >> seriously, I'll try to come up with an improved proposal over the next d= ays. > > Thus I tested on x86_64-linux the below. Ok for mainline? Yes, great!