Clojure

debug builds

Details

  • Type: Enhancement Enhancement
  • Status: Open Open
  • Priority: Minor Minor
  • Resolution: Unresolved
  • Affects Version/s: Release 1.2
  • Fix Version/s: Backlog
  • Component/s: None
  • Labels:
  • Approval:
    Vetted

Description

This ticket includes two patches:

  1. a patch to set assert when clojure.lang.RT loads, based on the presence of system property clojure.debug
  2. expand error messages in assert to include local-bindings</code> (a new macro which wraps the implicit <code>&env)

Things to consider before approving these patches:

  1. should there be an easy Clojure-level way to query if debug is enabled? (checking assert isn't the same, as debug should eventually drive other features)
  2. assertions will now be off by default – this is a change!
  3. is the addition of the name local-bindings to clojure.core cool?

Activity

Stuart Halloway made changes -
Field Original Value New Value
Summary turn-off-able asserts, with better messages debug builds
Assignee Rich Hickey [ richhickey ] Stuart Halloway [ stu ]
Fix Version/s Release.Next [ 10038 ]
Fix Version/s Backlog [ 10035 ]
Reporter Stuart Halloway [ stu ]
Priority Minor [ 4 ]
Approval Incomplete
Waiting On richhickey
Rich Hickey made changes -
Waiting On richhickey stu
Alexander Redington made changes -
Fix Version/s Approved Backlog [ 10034 ]
Fix Version/s Release.Next [ 10038 ]
Affects Version/s Release 1.2 [ 10037 ]
Stuart Halloway made changes -
Waiting On stu richhickey
Rich Hickey made changes -
Waiting On richhickey stu
Stuart Halloway made changes -
Waiting On stu richhickey
Stuart Halloway made changes -
Assignee Stuart Halloway [ stu ]
Alex Miller made changes -
Fix Version/s Approved Backlog [ 10034 ]
Fix Version/s Backlog [ 10035 ]
Alex Miller made changes -
Approval Incomplete [ 10006 ] Vetted [ 10003 ]
Alex Miller made changes -
Labels build

People

Vote (0)
Watch (4)

Dates

  • Created:
    Updated: