indexOf about bean HOT 10 CLOSED

Nope -- indexOf is only called on strings in bean. And as far as i know, indexOf on strings is supported in all major browsers.

from bean.

Also, for what it's worth -- bean will never be made to depend on any other microlibrary -- that would defeat the purpose. Thanks for bringing this up though! cheers :)

from bean.

actually -- you did point out a bug with this tough :D -- so i'll keep open...

from bean.

Noooo. Take a look at https://github.com/fat/bean/blob/master/bean.js#L79 and then at https://github.com/fat/bean/blob/master/bean.js#L264. Notice the trailing .split(',') in the latter ;)

from bean.

yeah that split shouldn't be there -- it's only splitting the last line :P, than + it to a string... it's a string. At any rate, i'm fixing that bit now

from bean.

alright -- this is now a string for sure... thanks again for bringing this up -- you rock!

from bean.

Cool.
Still, nothing pressing, but I have a feeling that (nativeEvents[type]) which is O(1) is much faster than nativeEvents.indexOf('|'+type+'|') which is O(length). Just a matter of smart one-time initialization of a hash from a string. What do you think? TIA

from bean.

yep that makes sense -- i'll change that now.

from bean.

pushed -- using your object lookup -- cheers

from bean.

Thanks!

from bean.