search

adept-dm / adept

Adept - the predictable dependency management system
171 stars 10 forks source link

Json #52

Closed aknuds1 closed 10 years ago

aknuds1 commented 10 years ago

I've rewritten JSON deserialization logic in a functional/declarative style, it looks much nicer than before, but maybe you'll spot more potential for improvement.

aknuds1 commented 10 years ago

Come to think of it, it might be wise to compare this scheme to the current (dirtier) one wrt. performance. While this looks nicer, maybe it's less performant?

freekh commented 10 years ago

Testing this now, but want this either way. Readability > perf