search

ros / genmsg

Standalone Python library for generating ROS message and service data structures for various languages
http://wiki.ros.org/genmsg
29 stars 74 forks source link

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

Closed cclauss closed 6 years ago

cclauss commented 6 years ago

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
dirk-thomas commented 6 years ago

Thank you for the patch.