AtnNn / librethinkdbxx

RethinkDB driver for C++
Other
98 stars 31 forks source link

write_datum in namespace RethinkDB is missing #29

Closed sushanttripathy closed 8 years ago

mbroadst commented 8 years ago

@sushanttripathy all the json-related methods were made internal, could you help describe your use case for needing this method publicly?

mbroadst commented 8 years ago

@sushanttripathy maybe you're looking for something like this: https://github.com/AtnNn/librethinkdbxx/blob/master/src/datum.cc#L410 ?

sushanttripathy commented 8 years ago

Hi Matt,

Thanks for the quick response. I was unaware of the change. :) I was merely trying to run the associated example in the readme markdown.

Best, Sushant

On Sat, Sep 3, 2016 at 6:10 AM, Matt Broadstone notifications@github.com wrote:

@sushanttripathy https://github.com/sushanttripathy maybe you're looking for something like this: https://github.com/AtnNn/ librethinkdbxx/blob/master/src/datum.cc#L410 ?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/AtnNn/librethinkdbxx/issues/29#issuecomment-244545531, or mute the thread https://github.com/notifications/unsubscribe-auth/AHMmkmHL9xa_G2vbdEyh3iEDQdFKjZawks5qmXHEgaJpZM4J0PY5 .

mbroadst commented 8 years ago

@sushanttripathy ah! thank you very much for that, I forgot to make the change there.