Fix missing lock guard - Githubissues

bitcoinjs / fast-dat-parser

Superfast blockchain parser for stats

MIT License

74 stars 26 forks source link

Fix missing lock guard #30

Closed NewProggie closed 4 years ago

NewProggie commented 4 years ago

This commit fixes a bug where the lock guard (for concurrently accessing the same scope from different threads) had basically no effect, due to being bound to a temporary only.

junderw commented 4 years ago

@fanatid Could you look at this?

fanatid commented 4 years ago

I had never read code of this project, but this changes looks reasonable. Object should be created on stack for locking.

junderw commented 4 years ago

Sounds good.