diff options
author | Ludovic Courtès <ludo@gnu.org> | 2022-03-07 10:29:27 +0100 |
---|---|---|
committer | Ludovic Courtès <ludo@gnu.org> | 2022-03-07 10:52:16 +0100 |
commit | 347321ece9fc85ddf74af3c798230b7b187fbce9 (patch) | |
tree | ec71cb2d4f077d4268ee95d7469c0f02a4b9f5df /test-suite/tests/compiler.test | |
parent | c572b11f3d1e5c663ce28bfd1410df53b5019ff3 (diff) | |
download | guile-347321ece9fc85ddf74af3c798230b7b187fbce9.tar.gz |
psyntax: Honor source properties for things other than syntax objects.
Commit 54bbe0b2846c5b1aa366c91d679ba724869c8cda inadvertently led
psyntax to dismiss source location info for data returned by read hash
extensions, because read hash extensions return plain data with
associated source properties, even when called from 'read-syntax'.
This change reverts part of this commit to restore that behavior.
Fixes <https://issues.guix.gnu.org/54003>.
* module/ice-9/psyntax.scm (datum-sourcev): New procedure.
(source-annotation): Fall back to 'datum-sourcev'.
* module/ice-9/psyntax-pp.scm: Regenerate.
* test-suite/tests/compiler.test ("psyntax")["syntax-source with
read-hash-extend"]: New test.
Diffstat (limited to 'test-suite/tests/compiler.test')
-rw-r--r-- | test-suite/tests/compiler.test | 26 |
1 files changed, 24 insertions, 2 deletions
diff --git a/test-suite/tests/compiler.test b/test-suite/tests/compiler.test index 466f2b821..d60151c6f 100644 --- a/test-suite/tests/compiler.test +++ b/test-suite/tests/compiler.test @@ -1,5 +1,5 @@ ;;;; compiler.test --- tests for the compiler -*- scheme -*- -;;;; Copyright (C) 2008-2014, 2018, 2021 Free Software Foundation, Inc. +;;;; Copyright (C) 2008-2014, 2018, 2021-2022 Free Software Foundation, Inc. ;;;; ;;;; This library is free software; you can redistribute it and/or ;;;; modify it under the terms of the GNU Lesser General Public @@ -19,6 +19,8 @@ #:use-module (test-suite lib) #:use-module (test-suite guile-test) #:use-module (system base compile) + #:use-module ((language tree-il) + #:select (tree-il-src call-args)) #:use-module ((system vm loader) #:select (load-thunk-from-memory)) #:use-module ((system vm program) #:select (program-sources source:addr))) @@ -70,7 +72,27 @@ (let ((m (make-module))) (beautify-user-module! m) (compile '(define round round) #:env m) - (eq? round (module-ref m 'round))))) + (eq? round (module-ref m 'round)))) + + (pass-if-equal "syntax-source with read-hash-extend" + '((filename . "sample.scm") (line . 2) (column . 5)) + ;; In Guile 3.0.8, psyntax would dismiss source properties added by + ;; read hash extensions on data they return. + ;; See <https://issues.guix.gnu.org/54003> + (with-fluids ((%read-hash-procedures + (fluid-ref %read-hash-procedures))) + (read-hash-extend #\~ (lambda (chr port) + (list 'magic (read port)))) + (tree-il-src + (car + (call-args + (call-with-input-string "\ +;; first line +;; second line + #~(this is a magic expression)" + (lambda (port) + (set-port-filename! port "sample.scm") + (compile (read-syntax port) #:to 'tree-il))))))))) (with-test-prefix "current-reader" |