Wiredcraft / carcass

A toolbox for Node.js.
http://wiredcraft.com/carcass/
MIT License
33 stars 8 forks source link

Change for loop to map. #46

Closed xeodou closed 10 years ago

makara commented 10 years ago

Thanks but I have a lot of for loops and we may want to change most of them to keep consistent.

Let me test with a benchmark first.

xeodou commented 10 years ago

Sure.

makara commented 10 years ago

Just realised that I already have a benchmark about that, built for mixin(). And for loop is actually faster.

Problem here is actually not the loops, but I don't need the return, and with coffee I need to return nothing explicitly. Changing that and closing this.