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

Fix undefined name: ‘l’ —> “orig_line‘ #75

Merged
merged 1 commit into from
Mar 13, 2018
Merged

Fix undefined name: ‘l’ —> “orig_line‘ #75

merged 1 commit into from
Mar 13, 2018

Commits on Mar 12, 2018

  1. Fix undefined name:: ‘l’ —> “orig_line‘ 

    __l__ is an undefined name in this context.  In the error message we want to show the user the original line.

flake8 testing of https://github.com/ros/genmsg on Python 2.7.14

$ __flake8 . --count --select=E901,E999,F821,F822,F823 --show-source --statistics__
```
./src/genmsg/msg_loader.py:206:69: F821 undefined name 'l'
            raise InvalidMsgSpec("Invalid constant declaration: %s"%l)
                                                                    ^
1     F821 undefined name 'l
```
    cclauss authored Mar 12, 2018
    Configuration menu
    Copy the full SHA
    fbae1dc View commit details
    Browse the repository at this point in the history