mdb_cursor_put params should be refs?

CoreyKaylor / Lightning.NET

.NET library for LMDB key-value store

Other

398 stars 82 forks source link

mdb_cursor_put params should be refs? #21

Closed hyc closed 10 years ago

hyc commented 10 years ago

I got a crash trying to use LightningCursor.Put(). Comparing the code to Transaction.Put(), it appears that the key/value params ought to be references. Yes? Not sure, haven't used C# much.

ilyalukyanov commented 10 years ago

Nice, thank you. It looks like yes. Haven't had enought time to review the pull request untill now. Now i merge it. But still no time to run tests. Hope to cover the cursors-related code with tests at nearest weekend.