Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Provide explicit Bson.toString() method. #468

Merged
merged 1 commit into from
Jan 17, 2014

Conversation

dnadlinger
Copy link
Contributor

Otherwise, opDispatch() will be invoked when trying to
writeln() a Bson object for debugging purposes, leading to
a confusing exception.

I'm not sure whether goign through Json is the best idea,
it just seemed like a convenient fix for the time being.

Otherwise, opDispatch() will be invoked when trying to
writeln() a Bson object for debugging purposes, leading to
a confusing exception.

I'm not sure whether goign through Json is the best idea,
it just seemed like a convenient fix for the time being.
@s-ludwig
Copy link
Member

That topic came up a number of times now and I always had to give an example with an explicit .toJson, so this actually makes a lot of sense. I think I was a bit too conservative here with adding methods to Bson. Going through JSON should be totally fine for now, since the output is meant purely for human inspection (no matching fromString).

s-ludwig added a commit that referenced this pull request Jan 17, 2014
Provide explicit Bson.toString() method.
@s-ludwig s-ludwig merged commit 0089cf6 into vibe-d:master Jan 17, 2014
@dnadlinger dnadlinger deleted the bson-tostring branch January 17, 2014 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants