Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Fix Pose2 print #685

Merged
merged 2 commits into from
Jan 25, 2021
Merged

Fix Pose2 print #685

merged 2 commits into from
Jan 25, 2021

Conversation

varunagrawal
Copy link
Collaborator

I made a mistake in the print method for Pose2 so fixed it and added a unit test.

Also updated all the empty string checks to use the standard empty() method.

@varunagrawal varunagrawal added the bugfix Fixes an issue or bug label Jan 25, 2021
@varunagrawal varunagrawal requested a review from ProfFan January 25, 2021 21:29
@varunagrawal varunagrawal self-assigned this Jan 25, 2021
@varunagrawal varunagrawal merged commit 364c437 into develop Jan 25, 2021
@varunagrawal varunagrawal deleted the fix/pose2-print branch January 25, 2021 23:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ProfFan ProfFan ProfFan approved these changes

Assignees

@varunagrawal varunagrawal

Labels
bugfix Fixes an issue or bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@varunagrawal @ProfFan