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

Replace NULL with nullptr #158

Merged
merged 1 commit into from
Aug 13, 2016
Merged

Replace NULL with nullptr #158

merged 1 commit into from
Aug 13, 2016

Conversation

jslee02
Copy link
Member

@jslee02 jslee02 commented Aug 13, 2016

for type safety

@sherm1
Copy link
Member

sherm1 commented Aug 13, 2016

👍

@jslee02 jslee02 merged commit e3cb308 into appveyor_win32 Aug 13, 2016
@mamoll
Copy link
Member

mamoll commented Aug 13, 2016

For more complex code transformations that take advantage of C++11 features, you should check out clang-tidy. See my commits on July 28 in the OMPL repo for examples.

@jslee02
Copy link
Member Author

jslee02 commented Aug 13, 2016

Good idea! I will take a look at clang-tidy once #154 is merged since the code transformation would be applied all over the code base. I wasn't patient with NULL until that. 😓

@jslee02 jslee02 deleted the nullptr branch August 13, 2016 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants