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

Jump to bottom

Rename doctest::detail::Result res; in DOCTEST_ASSERT_IMPLEMENT #10

Closed
iboB opened this issue May 25, 2016 · 0 comments
Closed

Rename doctest::detail::Result res; in DOCTEST_ASSERT_IMPLEMENT #10

iboB opened this issue May 25, 2016 · 0 comments

Comments

@iboB
Copy link
Member

iboB commented May 25, 2016

doctest::detail::Result res; takes up the quite common symbol name res and makes a test like CHECK(res == x); impossible.

Please rename it to something which would be much more unlikely to clash like __doctest__res__
onqtam added a commit that referenced this issue May 31, 2016
@onqtam
fixed #10
dba4723 
also added an example using the parts of the doctest header (and fixed it to work)
@onqtam onqtam closed this as completed in bc4b56c Sep 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@iboB