EJDB_EXPORT for bson_free #160

Merged
merged 1 commit into from Sep 13, 2015

Conversation

Projects
None yet
3 participants
@dmigous
Contributor

dmigous commented Aug 21, 2015

There are functions like bson2json() that allocates string, but bson_free() symbol isn't exported to library. Hence no way to free allocated memory and we need EJDB_EXPORT for bson_free()

Tests results:

100% tests passed, 0 tests failed out of 270
@dmigous

This comment has been minimized.

Show comment
Hide comment
@dmigous

dmigous Sep 11, 2015

Contributor

is anybody here?

Contributor

dmigous commented Sep 11, 2015

is anybody here?

@akaustel

This comment has been minimized.

Show comment
Hide comment
@akaustel

akaustel Sep 11, 2015

It would seem that Softmotions are on vacation or something like that. Not heard from anyone there in a while. I have a pull request on the ejdb-node repository waiting for acceptance. I'm waiting for another week or two until I get anxious.

It would seem that Softmotions are on vacation or something like that. Not heard from anyone there in a while. I have a pull request on the ejdb-node repository waiting for acceptance. I'm waiting for another week or two until I get anxious.

@dmigous

This comment has been minimized.

Show comment
Hide comment
@dmigous

dmigous Sep 11, 2015

Contributor

@akaustel thanks for response. Good to see someone alive here, hehe :)

Contributor

dmigous commented Sep 11, 2015

@akaustel thanks for response. Good to see someone alive here, hehe :)

@adamansky

This comment has been minimized.

Show comment
Hide comment
@adamansky

adamansky Sep 13, 2015

Member

Thank you very much! We've been on vacation. Now are busy days started so the next release with latest fixes will be landed within a few days.

Member

adamansky commented Sep 13, 2015

Thank you very much! We've been on vacation. Now are busy days started so the next release with latest fixes will be landed within a few days.

adamansky added a commit that referenced this pull request Sep 13, 2015

@adamansky adamansky merged commit 12de4c7 into Softmotions:master Sep 13, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment