<< Back to previous view

[CLJS-837] Cache namespace env Created: 08/Aug/14  Updated: 28/Dec/14  Resolved: 28/Dec/14

Status: Closed
Project: ClojureScript
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Enhancement Priority: Minor
Reporter: Nicola Mometto Assignee: Nicola Mometto
Resolution: Duplicate Votes: 1
Labels: None

Attachments: Text File 0001-add-support-for-cached-env.patch    


See: https://github.com/clojure/tools.analyzer.js/blob/master/src/main/clojure/clojure/tools/analyzer/js.clj#L524-L548

Comment by Nicola Mometto [ 11/Aug/14 8:09 AM ]

I've written an initial patch based on the same approach I used for t.a.js, it's attached as 0001-add-support-for-cached-env.patch
Currently I'm seeing an OOM exception when I invoke backup-env, after loading cljs.core, which is exactly what David told me he was getting when toying with a similar patch.

At a quick glance, it seems like the analyzer's ::namespace map contains significant more fields per var map than tools.analyzer.js's one, so that's probably why backup-env works fine for t.a.js but not for cljs

Comment by Nicola Mometto [ 28/Dec/14 12:49 PM ]

This is being addressed (or has already been addressed) by David in recent commits.

Generated at Sat Oct 10 04:07:15 CDT 2015 using JIRA 4.4#649-r158309.