[CLJ-939] Exceptions thrown in the top level ns form are reported without file or line number Created: 24/Feb/12 Updated: 15/Dec/12
|Affects Version/s:||Release 1.3, Release 1.4|
|Attachments:||0001-report-load-exceptions-with-file-and-line.diff 0002-report-load-exceptions-with-file-and-line.diff clj-939-report-load-exceptions-with-file-and-line-patch-v2.txt|
If there is an error in the `ns` form, an exception is thrown, which is not caught in `load`.
For example, with an invalid :only clause;
This generates a Don't know how to create ISeq from: clojure.lang.Symbol exception, with source file or line number.
|Comment by Hugo Duncan [ 25/Feb/12 8:26 AM ]|
|Comment by Andy Fingerhut [ 09/Mar/12 9:26 AM ]|
Patch 0001-report-load-exception-with-file-and-line.diff fails build. Patch 0002-report-load-exception-with-file-and-line.diff applies, builds, and tests cleanly as of March 9, 2012. Hugo has signed a CA.
|Comment by Andy Fingerhut [ 05/Oct/12 8:13 AM ]|
clj-939-report-load-exceptions-with-file-and-line-patch-v2.txt dated Oct 5 2012 is intended to be an update to Hugo Duncan's patch 0002-report-load-exceptions-with-file-and-line.diff dated Feb 25 2012. Because of Brandon Bloom's recently commited patch adding column numbers in addition to line numbers, this is not simply updating some lines of context, but I think it is correct. It would be good if Hugo could take a look at it and confirm.
|Comment by Stuart Sierra [ 09/Nov/12 9:38 AM ]|
The error messages are better than what we had before. The line/column numbers are not particularly informative, probably because ns is a macro.
|Comment by Rich Hickey [ 13/Nov/12 3:37 PM ]|
This patch doesn't change the reporting on any other (e.g. nested) exceptions? It looks like it might.