From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 29867 invoked by alias); 16 Aug 2008 06:43:51 -0000 Received: (qmail 29839 invoked by uid 22791); 16 Aug 2008 06:43:47 -0000 X-Spam-Check-By: sourceware.org Received: from wa-out-1112.google.com (HELO wa-out-1112.google.com) (209.85.146.177) by sourceware.org (qpsmtpd/0.31) with ESMTP; Sat, 16 Aug 2008 06:43:13 +0000 Received: by wa-out-1112.google.com with SMTP id k22so705914waf.20 for ; Fri, 15 Aug 2008 23:43:11 -0700 (PDT) Received: by 10.114.75.1 with SMTP id x1mr3487110waa.25.1218868991821; Fri, 15 Aug 2008 23:43:11 -0700 (PDT) Received: by 10.114.113.11 with HTTP; Fri, 15 Aug 2008 23:43:11 -0700 (PDT) Message-ID: <206fcf960808152343o4c8777d8w2271300ceac532fb@mail.gmail.com> Date: Sat, 16 Aug 2008 07:45:00 -0000 From: "Gabriel Dos Reis" To: "Aldy Hernandez" Subject: Re: [PATCH] caret diagnostics (was: broken FE diagnostics wrt complex expressions) Cc: "Manuel L?pez-Ib??ez" , "Joseph S. Myers" , "Tom Tromey" , dberlin@dberlin.org, jakub@redhat.com, gcc@gcc.gnu.org, "Chris Lattner" , "Gcc Patch List" In-Reply-To: <20080814171413.GA4460@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline References: <6c33472e0808140340o2a56b9f4n149025bfa8426331@mail.gmail.com> <6c33472e0808140527i158255d9hdbc677bd0870cbc1@mail.gmail.com> <6c33472e0808140556y1f8975boaf41babaeec39f98@mail.gmail.com> <6c33472e0808140741hb83b278jad9b1a5b15e8ab02@mail.gmail.com> <20080814150810.GA22079@redhat.com> <6c33472e0808140816w45e17254p1cb769044a6f284b@mail.gmail.com> <20080814171413.GA4460@redhat.com> X-Google-Sender-Auth: 9d07fa6e728360bf 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: 2008-08/txt/msg01085.txt.bz2 On Thu, Aug 14, 2008 at 12:14 PM, Aldy Hernandez wrote: >> * In the near future, make -fdiagnostics-show-caret the default at >> least while in experimental mode or at least during stages1 and 2. >> When making a release -fno-diagnostics-show-caret would be the >> default. Do this through a configure option that sets the default. >> >> * In the far away future, review the defaults and get rid of the >> configure option. > > To be honest, I don't mind either way. Either your option or Joseph's > is fine. I am however interested in getting the caret diagnostics in, > and then I can work on location information that will benefit everyone. I'm in favor of getting -fdiagnostics-show-caret=no by default in this release, and enable people like you to get useful stuff done. That gives us time to iron out outstanding bugs for the next release (and making it the default). -- Gaby > > Aldy >