search

shaneMangudi / bcrypt-nodejs

Native implementation of bcrypt for NodeJS
Other
574 stars 69 forks source link

should rewrite getByte(c) without try-catch #38

Open MetaMemoryT opened 9 years ago

MetaMemoryT commented 9 years ago

https://github.com/shaneGirish/bcrypt-nodejs/blob/master/bCrypt.js#L235

http://jsperf.com/try-catch-performance-overhead

and it pollutes debugging with extra exceptions being thrown

fpirsch commented 9 years ago

If you want performance, maybe you should take a look at Twin-Bcrypt.