Synchronous Open Support #48

Merged
merged 2 commits into from Feb 7, 2013

Projects

None yet

2 participants

@ulikoehler

I finally managed to get a version of syncOpen() working -- this has already been requested in issue #26.

The behaviour is exactly equal to the async version, I just put all the code from OpenAsync into the new static function syncOpen. It throws an exception on error.

I also wrote a rudimentary unit test for syncOpen() that simply tests if the handle returned passes a simple put/get test.

Merging would be highly appreciated as this feature has been requested repeatedly.

@my8bird my8bird merged commit fcd6f72 into my8bird:master Feb 7, 2013
@my8bird
Owner
my8bird commented Feb 7, 2013

Sorry, that took to so long. I had a funeral and lost track for a bit. Thanks for the great update.

@ulikoehler

4 days is really fast IMO ;-) Thanks for merging!

@ulikoehler ulikoehler deleted the unknown repository branch Feb 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment