[CLJS-177] js->clj should convert JavaScript null to Clojure nil

Tue, 10 Apr 2012 19:38:02 -0500

Prior to commit ebf45f5 [A], js->clj converted JavaScript null to Clojure nil without any errors.

As of commit ebf45f5 (related to ~~CLJS-175~~), js->clj throws a TypeError when it encounters JavaScript null:

TypeError: Cannot read property 'constructor' of null
b===c}function q(b,c){return gb.call(e,b,c)}function pb(b){return b.construct
^
TypeError: Cannot read property 'constructor' of null

The attached patch [B] adds a test to demonstrate the issue and provides a suggested fix.

Note: ~~CLJS-175~~ intentionally limited the types of objects it would attempt to convert to Clojure code. For example, ~~CLJS-175~~ notes that conversion "doesn't make sense for something like Date or RegExp." I agree for those types of objects, but "null" seems straightforward enough that it should be converted.

[A] https://github.com/clojure/clojurescript/commit/ebf45f5
[B] Also viewable here: https://github.com/jasonrudolph/clojurescript/compare/master...js-null-to-clj-nil

Clojure JIRA

[CLJS-177] js->clj should convert JavaScript null to Clojure nil