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.

Sign up for GitHub

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

Fix compile issue in people_app #3618

Merged
merged 1 commit into from
Jan 31, 2020
Merged

Fix compile issue in people_app #3618

merged 1 commit into from
Jan 31, 2020

Conversation

SunBlack
Copy link
Contributor

It was ambiguous because of using namespace std and pcl at same time. To prevent such issue using namespace is now removed.

This fixes #3617

Fix compile issue in people_app.cpp as shared_ptr was ambiguous.
923bced 
It was ambiguous because of using namespace std and pcl at same time. To prevent such issue `using namespace` is now removed.
@kunaltyagi
Copy link
Member

LGTM, Waiting on CI

@taketwo taketwo merged commit 145e138 into PointCloudLibrary:master Jan 31, 2020
@SunBlack SunBlack deleted the fix_people_app branch February 1, 2020 12:19
@taketwo taketwo changed the title Fix compile issue in people_app.cpp as shared_ptr was ambiguous. Fix compile issue in people_app Mar 18, 2020
@taketwo taketwo added the changelog: fix Meta-information for changelog generation label Mar 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@taketwo taketwo taketwo approved these changes

Assignees
No one assigned
Labels
changelog: fix Meta-information for changelog generation module: gpu
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

GPU/people_app is currently not compiling
3 participants
@SunBlack @kunaltyagi @taketwo