warpech / sheetclip

Copy/paste from your HTML5 web app to a spreadsheet
MIT License
304 stars 27 forks source link

void 0 #1

Closed rwaldron closed 11 years ago

rwaldron commented 11 years ago

https://github.com/warpech/sheetclip/blob/master/sheetclip.js#L16-17

Why?

Use void 0

warpech commented 11 years ago

You are right, void 0 is probably the best way to do it as I just checked at http://jsperf.com/type-of-undefined-vs-undefined/6

I didn't put much thought into that. I guess there are better optimizations to be made in the code :)

i will update it next time I touch the code.

warpech commented 11 years ago

Also, big thanks for your article about Object.observe! Lack of it gives me headaches while working on my other project, https://github.com/warpech/angular-patch

rwaldron commented 11 years ago

My pleasure :)