unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Steve Kargl <sgk@troutmask.apl.washington.edu>
Cc: wt@atmos.colostate.edu, fortran@gcc.gnu.org, emacs-devel@gnu.org
Subject: Re: Emacs and GFortran
Date: Wed, 1 Nov 2006 14:35:48 -0800	[thread overview]
Message-ID: <20061101223548.GA72295@troutmask.apl.washington.edu> (raw)
In-Reply-To: <20061101215142.51A0944013@Psilocybe.Update.UU.SE>

On Wed, Nov 01, 2006 at 10:51:42PM +0100, Alfred M. Szmidt wrote:
>    Have you actually read gfortran source code other than error.c?
> 
> I have.

Good, then you can appreciate that the problem isn't trivial.

> 
>    > Here is another version that is should be backward compatible
>    > with the bits that the testsuite parses.  This will fix the
>    > problem the original user reported; namley allowing programs that
>    > parse GNU style error message to jump to the location of the
>    > error/warning.
> 
>    I've already posted a counter example that will not allow a
>    person to jump to the location of the error.  You need to 
>    read the entire error message to decide where the error is.
> 
> Your example doesn't counter anything, this is your example with GNU
> style error messages (simply removing "In file", and adding a extra
> semi-colon):
> 
> | l.inc:1:
> | 
> |      Included at l.f90:2
> | 
> |    integer i
> |            1
> | l.f90:3:
> | 
> |    call i(j)
> |            2
> | Error: 'i' at (1) has a type, which is not consistent with the CALL at (2)
> 
> Which will work with any program that parses GNU style error messages
> by producing two places where one can go to; the user can choose which
> place to edit.  It doesn't have to catch the exact place, it is after
> all not a substitute for a brain.

What about the third location?  You need to parse "Included at l.f90:2"
to jump to that location to remove the included file.

> Can you please apply one of those patches?  They both fix the problem.

But it does not fix the nonconformance to the GNU standard.  The above 
message would need to become

l.f90:3-l.inc:1: error: 'i' has a type, which is not consistent with the CALL

or

l.f90:3.7-l.inc:1.3: error: 'i' has a type, which is not consistent with the CALL

In neither case, do we see the problem is actually at line 2 of l.f90.

-- 
Steve


  reply	other threads:[~2006-11-01 22:35 UTC|newest]

Thread overview: 88+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-30 10:32 Emacs and GFortran Angelo Graziosi
2006-10-30 14:15 ` Stefan Monnier
2006-10-30 15:27   ` Angelo Graziosi
2006-10-30 15:38     ` Stefan Monnier
2006-10-30 16:14       ` Steve Kargl
2006-10-30 18:28         ` Stefan Monnier
2006-10-30 20:35           ` Angelo Graziosi
2006-10-30 21:00             ` Tobias Burnus
2006-10-31  8:32               ` Angelo Graziosi
2006-10-31 10:54               ` Angelo Graziosi
2006-10-31 16:06                 ` Stuart D. Herring
2006-10-31 23:24                   ` Angelo Graziosi
2006-10-30 21:16             ` Nick Roberts
2006-10-30 19:16 ` Richard Stallman
2006-10-30 21:13   ` Steve Kargl
2006-10-31  4:02     ` Miles Bader
2006-10-31  4:31     ` Eli Zaretskii
2006-10-31  4:31       ` Steve Kargl
2006-10-31 22:07         ` Eli Zaretskii
2006-11-01  2:13         ` Richard Stallman
2006-11-01  2:16           ` Miles Bader
2006-10-31  6:11     ` Stefan Monnier
2006-10-31  6:43       ` FX Coudert
2006-11-01  2:14         ` Richard Stallman
2006-11-01  2:23           ` Miles Bader
2006-10-31  8:26     ` Angelo Graziosi
2006-10-31 15:10       ` Steve Kargl
2006-10-31 15:47         ` Angelo Graziosi
2006-10-31 16:00           ` Chong Yidong
2006-11-01 16:40           ` Richard Stallman
2006-11-01 17:37             ` Warren Turkal
2006-11-01 18:03               ` Steve Kargl
2006-11-01 20:37                 ` Stefan Monnier
2006-11-01 20:39                 ` Alfred M. Szmidt
2006-11-01 21:30                   ` Steve Kargl
2006-11-01 21:51                     ` Alfred M. Szmidt
2006-11-01 22:35                       ` Steve Kargl [this message]
2006-11-01 22:49                         ` Alfred M. Szmidt
2006-11-01 23:26                           ` Steve Kargl
2006-11-01 23:35                             ` Alfred M. Szmidt
2006-11-02  3:12                               ` Steve Kargl
2006-11-03  2:40                                 ` Chong Yidong
2006-11-03  3:32                                   ` Steve Kargl
2006-11-03 10:05                                   ` David Kastrup
2006-11-03  2:44                                 ` Chong Yidong
2006-11-02 13:26                             ` Alfred M. Szmidt
2006-11-02 19:00                               ` Thomas Koenig
2006-11-02 20:22                                 ` Eli Zaretskii
2006-11-02 20:32                                   ` David Kastrup
2006-11-03 13:26                                     ` Eli Zaretskii
2006-11-02 20:38                                   ` Thomas Koenig
2006-11-02 20:43                                   ` Steve Kargl
2006-11-03 12:30                                     ` Eli Zaretskii
2006-11-03 12:58                                       ` David Kastrup
2006-11-03 13:40                                         ` Eli Zaretskii
2006-11-02 22:27                                   ` François-Xavier Coudert
2006-11-03 12:51                                     ` Eli Zaretskii
2006-11-03  0:43                                   ` Steven Bosscher
2006-11-03  1:08                                     ` Alfred M. Szmidt
2006-11-03  1:21                                       ` Steve Kargl
2006-11-03  2:32                                         ` Brooks Moses
2006-11-03 18:59                                           ` Nick Roberts
2006-11-03 20:27                                             ` Brooks Moses
2006-11-15  4:09                                             ` Brooks Moses
2006-11-15  4:30                                               ` Nick Roberts
2006-11-15  5:09                                                 ` Brooks Moses
2006-11-15  7:57                                                   ` Nick Roberts
2006-11-15  9:03                                                     ` David Kastrup
2006-11-15  8:43                                                   ` David Kastrup
2006-11-15  9:14                                                     ` Andrew Pinski
2006-11-15  9:16                                                       ` David Kastrup
2006-11-16  6:23                                                       ` Richard Stallman
2006-11-03  7:06                                       ` Steven Bosscher
2006-11-03 13:10                                         ` Eli Zaretskii
2006-11-04  6:37                                       ` Richard Stallman
2006-11-03 13:05                                     ` Eli Zaretskii
2006-10-31 23:01         ` Eli Zaretskii
2006-11-02 22:20           ` Paul Thomas
2006-11-03 12:45             ` Eli Zaretskii
2006-11-03 13:14               ` David Kastrup
2006-11-03 13:42                 ` Eli Zaretskii
     [not found] <45462048.8020102@net-b.de>
     [not found] ` <20061030161402.15694.qmail@web81209.mail.mud.yahoo.com>
2006-10-30 16:18   ` François-Xavier Coudert
2006-10-31  1:59     ` Miles Bader
2006-10-31  2:46       ` Steve Kargl
2006-10-31  3:49         ` Miles Bader
2006-10-31  4:23           ` Steve Kargl
2006-10-31  5:39         ` stephen
2006-10-31 16:40         ` Alfred M. Szmidt

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=20061101223548.GA72295@troutmask.apl.washington.edu \
    --to=sgk@troutmask.apl.washington.edu \
    --cc=emacs-devel@gnu.org \
    --cc=fortran@gcc.gnu.org \
    --cc=wt@atmos.colostate.edu \
    /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.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

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).