ClojureScript

Include line and file information in error messages

Details

  • Type: Enhancement Enhancement
  • Status: Resolved Resolved
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
  • Patch:
    Code

Description

Just as the ana/warning function did, now errors and assertions include line and file source information. I needed this sorely.

Attachments

  1. Text File
    CLJS-432-v001.patch
    28/Nov/12 5:48 PM
    15 kB
    Brandon Bloom
  2. Text File
    CLJS-432-v002.patch
    28/Nov/12 6:13 PM
    6 kB
    Brandon Bloom
  3. Text File
    CLJS-432-v003.patch
    29/Nov/12 2:28 PM
    6 kB
    Brandon Bloom
  4. Text File
    CLJS-432-v004.patch
    29/Nov/12 2:50 PM
    2 kB
    Brandon Bloom
  5. Text File
    CLJS-432-v005.patch
    29/Nov/12 2:57 PM
    2 kB
    Brandon Bloom
  6. Text File
    CLJS-432-v006.patch
    29/Nov/12 5:05 PM
    4 kB
    Brandon Bloom

Activity

Descending order - Click to sort in ascending order
Hide
Permalink
Brandon Bloom added a comment -

v6 improves errors in repl as discussed in IRC

Show
Brandon Bloom added a comment - v6 improves errors in repl as discussed in IRC
Hide
Permalink
Brandon Bloom added a comment -

v5 also catches error from parse-invoke

Show
Brandon Bloom added a comment - v5 also catches error from parse-invoke
Hide
Permalink
Brandon Bloom added a comment -

v4 of patch as discussed in irc

Show
Brandon Bloom added a comment - v4 of patch as discussed in irc
Hide
Permalink
Brandon Bloom added a comment -

v3 of patch fixes issue with v2

Show
Brandon Bloom added a comment - v3 of patch fixes issue with v2
Hide
Permalink
David Nolen added a comment -

I don't have a problem with the approach in the first patch. I don't really see how a less invasive patch is even possible - you still need to pass the environment to some assertion check if you are going to throw a custom exception as well.

Show
David Nolen added a comment - I don't have a problem with the approach in the first patch. I don't really see how a less invasive patch is even possible - you still need to pass the environment to some assertion check if you are going to throw a custom exception as well.
Hide
Permalink
Brandon Bloom added a comment -

That v2 was made hastily. Upon further thought, it seems broken in the face of recursive analysis. I think the right thing, if we prefer the exception catching approach, is to rethrow as a custom exception type, which would be allowed through un-re-wrapped.

For example:

(try
  (parse-form ...)
  (catch AnalysisError e
    throw e)
  (catch Throwable e
    (throw (AnalysisError. env e))))
Show
Brandon Bloom added a comment - That v2 was made hastily. Upon further thought, it seems broken in the face of recursive analysis. I think the right thing, if we prefer the exception catching approach, is to rethrow as a custom exception type, which would be allowed through un-re-wrapped. For example: 
(try
  (parse-form ...)
  (catch AnalysisError e
    throw e)
  (catch Throwable e
    (throw (AnalysisError. env e))))
Hide
Permalink
Brandon Bloom added a comment -

Added v2 of patch that will handle any exception during parsing. Similar could be done during code generation. This approach seems much more robust and less invasive.

Show
Brandon Bloom added a comment - Added v2 of patch that will handle any exception during parsing. Similar could be done during code generation. This approach seems much more robust and less invasive.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4#649-r158309) | Report a problem
Powered by a free Atlassian JIRA open source license for Clojure. Try JIRA - bug tracking software for your team.