ClojureScript

Comparing a js/Date to nil throws

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: None
  • Labels:
  • Patch:
    Code

Description

(= (js/Date.) nil)

  1. date-equiv.patch
    05/Jun/12 9:28 PM
    0.7 kB
    Brandon Bloom
  2. date-equiv2.patch
    17/Jun/12 4:28 PM
    3 kB
    Brandon Bloom
  3. date-equiv3.patch
    18/Jun/12 4:45 PM
    2 kB
    Brandon Bloom

Activity

Hide
Brandon Bloom added a comment -

Same as patch 2 with commits squashed

Show
Brandon Bloom added a comment - Same as patch 2 with commits squashed
Hide
David Nolen added a comment -

Thanks can we collapse the patch into one commit?

Show
David Nolen added a comment - Thanks can we collapse the patch into one commit?
Hide
Brandon Bloom added a comment -

Sorry, forgot about this ticket. Attached patch has two additional commits: 1st stops calling .getTime with list/EMPTY via unecessary '() and the 2nd moves instance? from the "preds" to "fundamentals" section of core.cljs.

Show
Brandon Bloom added a comment - Sorry, forgot about this ticket. Attached patch has two additional commits: 1st stops calling .getTime with list/EMPTY via unecessary '() and the 2nd moves instance? from the "preds" to "fundamentals" section of core.cljs.
Hide
David Nolen added a comment -

updated patch please.

Show
David Nolen added a comment - updated patch please.
Hide
Brandon Bloom added a comment -

D'oh! Sorry, this patch causes a warning:

WARNING: Use of undeclared Var cljs.core/instance? at line 384

Show
Brandon Bloom added a comment - D'oh! Sorry, this patch causes a warning: WARNING: Use of undeclared Var cljs.core/instance? at line 384

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: