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

Fixed indentation. #18

Closed

Conversation

Projects
None yet
2 participants
@Cakem1x
Copy link

commented Nov 24, 2014

Separate pull request with just the indentation fixes in the face_detection.cpp

@dlaz

This comment has been minimized.

Copy link
Member

commented Nov 25, 2014

Sorry to keep going back and forth about this - the indentation is now a mix of spaces and tabs which is not ideal. The standard in ROS is two spaces. roslint has an example of using astyle to do this automatically. If you could fix the tab issue, that would be great. If you want to go ahead and run astyle on all the C++ and header files, that would also be a welcome change.

dlaz added a commit that referenced this pull request Dec 10, 2014

@dlaz

This comment has been minimized.

Copy link
Member

commented Dec 10, 2014

I ran the entire metapackage through astyle to cleanup the formatting.

@dlaz dlaz closed this Dec 10, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.