public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
From: "Manuel López-Ibáñez" <lopezibanez@gmail.com>
To: "Joseph S. Myers" <joseph@codesourcery.com>
Cc: "Tom Tromey" <tromey@redhat.com>,
	"Aldy Hernandez" <aldyh@redhat.com>,
	 	dberlin@dberlin.org, jakub@redhat.com, gcc@gcc.gnu.org,
	 	gdr@integrable-solutions.net,
	"Chris Lattner" <clattner@apple.com>,
	 	"Gcc Patch List" <gcc-patches@gcc.gnu.org>
Subject: Re: [PATCH] caret diagnostics (was: broken FE diagnostics wrt complex expressions)
Date: Thu, 14 Aug 2008 15:03:00 -0000	[thread overview]
Message-ID: <6c33472e0808140741hb83b278jad9b1a5b15e8ab02@mail.gmail.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0808141338160.25519@digraph.polyomino.org.uk>

2008/8/14 Joseph S. Myers <joseph@codesourcery.com>:
>
> But in any case the default should be the default with no configure
> option, users liking it should find their makefiles work the same
> everywhere and users not liking it can add the opposite option.

Then we are not going to get correct locations ever. New users do not
read the manual. Neither old users do. New functionality disabled by
default will be lost for both. I am fairly sure that a significant
percentage of GCC developers (not just users) do not know about
-fdiagnostics-show-option.

But even more importantly. No GCC developer is going to explicitly
enable caret diagnostics while developing GCC. How many nowadays use
-fshow-column or -fdiagnostics-show-option to check locations?

Moreover, caret diagnostics was mentioned as the way to solve the PRs
that Aldy mentioned. If it is disabled by default, how does it solve
anything? Why bother? I would really feel that I contributed to make
GCC worse if GCC diagnostics are less expressive because we have the
excuse of "you could enable the caret". I feel that a lot of PRs that
request for better diagnostics would be closed that way.

I feel that this thread is going again the same way as the ones
before. Someone says: Hey, having caret diagnostics would solve a lot
of problems! Everybody says: Yeah, that would be cool! We could do
this and that and all kinds of cool things. Then someone says: Oh yes
but we need to solve all these boring things that nobody ever really
looks and it should be disabled by default. Then one year later
someone says: Hey, having caret diagnostics would solve a lot of
problems!

Cheers,

Manuel.

  reply	other threads:[~2008-08-14 14:41 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-14 12:12 Manuel López-Ibáñez
2008-08-14 12:41 ` Joseph S. Myers
2008-08-14 12:57   ` Manuel López-Ibáñez
2008-08-14 13:54     ` Joseph S. Myers
2008-08-14 14:07       ` Manuel López-Ibáñez
2008-08-14 14:41         ` Joseph S. Myers
2008-08-14 15:03           ` Manuel López-Ibáñez [this message]
2008-08-14 15:08             ` [PATCH] caret diagnostics Robert Dewar
2008-08-14 15:22               ` Joseph S. Myers
2008-08-14 15:43                 ` Robert Dewar
2008-08-14 15:48               ` Manuel López-Ibáñez
2008-08-14 16:06                 ` Robert Dewar
2008-08-14 16:18                   ` Joseph S. Myers
2008-08-14 17:22                     ` Chris Lattner
2008-08-16 13:30                       ` Paolo Bonzini
2008-08-16 17:19                         ` Joseph S. Myers
2008-08-16 18:59                           ` Paolo Bonzini
2008-08-14 16:18                   ` Manuel López-Ibáñez
2008-08-14 18:49                 ` Mark Mitchell
2008-08-14 19:10                   ` Manuel López-Ibáñez
2008-08-14 19:28                     ` Mark Mitchell
2008-08-14 15:17             ` [PATCH] caret diagnostics (was: broken FE diagnostics wrt complex expressions) Joseph S. Myers
2008-08-14 17:18               ` [PATCH] caret diagnostics Tom Tromey
2008-08-14 17:21                 ` Ralf Wildenhues
2008-08-14 17:34                   ` Joseph S. Myers
2008-08-14 18:20                   ` Tom Tromey
2008-08-14 18:34                     ` Ralf Wildenhues
2008-08-14 19:02                       ` Robert Dewar
2008-08-14 17:33                 ` Joseph S. Myers
2008-08-14 17:51                 ` Joseph S. Myers
2008-08-14 18:52                   ` Mark Mitchell
2008-08-16 10:09                 ` Gabriel Dos Reis
2008-08-16 13:00                   ` Robert Dewar
2008-08-14 15:18             ` [PATCH] caret diagnostics (was: broken FE diagnostics wrt complex expressions) Aldy Hernandez
2008-08-14 15:29               ` Manuel López-Ibáñez
2008-08-14 17:23                 ` Aldy Hernandez
2008-08-16  7:45                   ` Gabriel Dos Reis
2008-08-16 13:12                     ` [PATCH] caret diagnostics Robert Dewar
2008-08-16  7:44       ` [PATCH] caret diagnostics (was: broken FE diagnostics wrt complex expressions) Gabriel Dos Reis
2008-08-14 18:00 ` [PATCH] caret diagnostics Tom Tromey

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6c33472e0808140741hb83b278jad9b1a5b15e8ab02@mail.gmail.com \
    --to=lopezibanez@gmail.com \
    --cc=aldyh@redhat.com \
    --cc=clattner@apple.com \
    --cc=dberlin@dberlin.org \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gcc@gcc.gnu.org \
    --cc=gdr@integrable-solutions.net \
    --cc=jakub@redhat.com \
    --cc=joseph@codesourcery.com \
    --cc=tromey@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).