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

Mutable Default Arugments #60

Open

BillWSY opened this issue Jul 23, 2018 · 0 comments

BillWSY commented Jul 23, 2018 •

edited

Loading

executive_smach/smach/src/smach/container.py

Line 122 in 1a44c12

def register_start_cb(self, start_cb, cb_args=[]):

https://github.com/google/styleguide/blob/gh-pages/pyguide.md#2123-cons

This particular case here might not cause any issue, but still, it is a terrible programming style and a common issue for Python beginners.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment