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

Better message printing for pcd_viewer #744

Conversation

xiangxw
Copy link
Contributor

@xiangxw xiangxw commented Jun 12, 2014

Camera file restore message will print after the file is loaded.

Camera file restore message will print after the file is loaded.
@taketwo
Copy link
Member

taketwo commented Jun 12, 2014

Looks good to me. Thanks!

taketwo added a commit that referenced this pull request Jun 12, 2014
…ssage

Better message printing for pcd_viewer
@taketwo taketwo merged commit a2be3ea into PointCloudLibrary:master Jun 12, 2014
@xiangxw xiangxw deleted the pcd_viewer_print_auto_restore_message branch June 12, 2014 11:59
@@ -122,6 +122,7 @@ pcl::visualization::PCLVisualizer::PCLVisualizer (const std::string &name, const
, shape_actor_map_ (new ShapeActorMap)
, coordinate_actor_map_ (new CoordinateActorMap)
, camera_set_ ()
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hi @taketwo just a question about C++, does this line set camera_set_ to false or let camera_set_ uninitialized? Thanks.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

camera_set_ will be set to false in this case. See this SO answer for details.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot. @taketwo

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