Consider using lazy-require #190

Closed
opened 6 years ago by sorawee · 1 comments
sorawee commented 6 years ago (Migrated from github.com)

I tried:

(lazy-require
 ["../render.rkt" (render-batch)])

in command.rkt instead of simply (require "../render.rkt"). This simple change consistently speeds up raco pollen reset (and other commands that don't render) by 0.6s. I think there are a lot more low hanging fruit that we can find.

I tried: ``` (lazy-require ["../render.rkt" (render-batch)]) ``` in `command.rkt` instead of simply `(require "../render.rkt")`. This simple change consistently speeds up `raco pollen reset` (and other commands that don't render) by 0.6s. I think there are a lot more low hanging fruit that we can find.
mbutterick commented 6 years ago (Migrated from github.com)

I didn’t even know about lazy-require so I imagine you’re right.

I didn’t even know about `lazy-require` so I imagine you’re right.
Sign in to join this conversation.
No Label
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: mbutterick/pollen#190
Loading…
There is no content yet.