unofficial mirror of bug-guile@gnu.org 
 help / color / mirror / Atom feed
* bug#26107: Inaccurate location info for unbound-variable errors
@ 2017-03-15 14:03 Ludovic Courtès
  2021-05-18 18:33 ` Taylan Kammer
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2017-03-15 14:03 UTC (permalink / raw)
  To: 26107

Hi!

In 2.1.8/2.2.0, walking up the stack to find the origin of an unbound
variable error yields inaccurate location info.  Consider this:

--8<---------------cut here---------------start------------->8---
(list 'a
      (symbol->string 'b)
      (symbol->string 'c)  ;← unbound variable error reported here
      (FOO 'bar)           ;… instead of here
      3
      4)
--8<---------------cut here---------------end--------------->8---

We get:

--8<---------------cut here---------------start------------->8---
$ guild compile -t value t.scm
Backtrace:
          18 (apply-smob/1 #<catch-closure 1312aa0>)
In ice-9/boot-9.scm:
    710:2 17 (call-with-prompt _ _ #<procedure default-prompt-handler (k proc)>)
In ice-9/eval.scm:
    619:8 16 (_ #(#(#<directory (guile-user) 13ba1e0>)))
In /gnu/store/4xir0dlzxl7h5v7va9rz8x3mvarlv2fd-profile/bin/guild:
    72:17 15 (main _)
In srfi/srfi-1.scm:
    640:9 14 (for-each #<procedure 128be40 at scripts/compile.scm:248:14 (file)> ("t.scm"))
In scripts/compile.scm:
   251:26 13 (_ _)
In system/base/target.scm:
     57:6 12 (with-target _ _)
In system/base/compile.scm:
    152:6 11 (compile-file "t.scm" #:output-file _ #:from _ #:to _ #:env _ #:opts _ #:canonicalization _)
     43:4 10 (call-once _)
In ice-9/boot-9.scm:
    846:4  9 (with-throw-handler _ _ _)
In system/base/compile.scm:
    59:11  8 (_)
   155:11  7 (_ #<closed: file 160a0e0>)
   224:14  6 (read-and-compile _ #:from _ #:to _ #:env _ #:opts _)
    255:6  5 (compile _ #:from _ #:to _ #:env _ #:opts _)
   183:32  4 (compile-fold _ #<tree-il (call (toplevel list) (const a) (call (toplevel symbol->string) (const b)) (call (toplevel symbol->string) (const c)) (call (toplevel FO…> …)
In ice-9/boot-9.scm:
   2309:4  3 (save-module-excursion #<procedure 1345d60 at language/bytecode/spec.scm:33:9 ()>)
In language/bytecode/spec.scm:
    35:19  2 (_)
In t.scm:
      3:6  1 (_)       ;← incorrect line number! (expected line 4)
In ice-9/boot-9.scm:
   757:25  0 (dispatch-exception _ _ _)

ice-9/boot-9.scm:757:25: In procedure dispatch-exception:
ice-9/boot-9.scm:757:25: In procedure module-lookup: Unbound variable: FOO
--8<---------------cut here---------------end--------------->8---

It Would Be Nice if we could get the right location info here, though I
understand this is a corner case.

Thoughts?

Ludo’.





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#26107: Inaccurate location info for unbound-variable errors
  2017-03-15 14:03 bug#26107: Inaccurate location info for unbound-variable errors Ludovic Courtès
@ 2021-05-18 18:33 ` Taylan Kammer
  2021-05-18 20:11   ` Ludovic Courtès
  0 siblings, 1 reply; 4+ messages in thread
From: Taylan Kammer @ 2021-05-18 18:33 UTC (permalink / raw)
  To: 26107, Ludovic Courtès

FWIW this seems fixed in 3.0, but I can reproduce with 2.2.7.

Are we still supporting 2.2 for non-critical stuff?

-- 
Taylan





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#26107: Inaccurate location info for unbound-variable errors
  2021-05-18 18:33 ` Taylan Kammer
@ 2021-05-18 20:11   ` Ludovic Courtès
  2021-05-18 20:12     ` Taylan Kammer
  0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2021-05-18 20:11 UTC (permalink / raw)
  To: Taylan Kammer; +Cc: 26107

Hi,

Taylan Kammer <taylan.kammer@gmail.com> skribis:

> FWIW this seems fixed in 3.0, but I can reproduce with 2.2.7.
>
> Are we still supporting 2.2 for non-critical stuff?

There’s no plan to do so.

Ludo’.





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#26107: Inaccurate location info for unbound-variable errors
  2021-05-18 20:11   ` Ludovic Courtès
@ 2021-05-18 20:12     ` Taylan Kammer
  0 siblings, 0 replies; 4+ messages in thread
From: Taylan Kammer @ 2021-05-18 20:12 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: 26107-done

On 18.05.2021 22:11, Ludovic Courtès wrote:
> Hi,
> 
> Taylan Kammer <taylan.kammer@gmail.com> skribis:
> 
>> FWIW this seems fixed in 3.0, but I can reproduce with 2.2.7.
>>
>> Are we still supporting 2.2 for non-critical stuff?
> 
> There’s no plan to do so.
> 
> Ludo’.
> 

Closing then. :-)

-- 
Taylan





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2021-05-18 20:12 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-15 14:03 bug#26107: Inaccurate location info for unbound-variable errors Ludovic Courtès
2021-05-18 18:33 ` Taylan Kammer
2021-05-18 20:11   ` Ludovic Courtès
2021-05-18 20:12     ` Taylan Kammer

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