cl showcxxmacros #523
cl showcxxmacros #523
Conversation
|
Sider detected 1 error on analyzing the commit ebc86a0.
We recommend fixing them as possible by updating the dependencies, configuring the analysis tool, configuring If you have problems or questions still, feel free to ask us via chat. 💬 You can turn off this notification if you don't want to receive it from now on. |
|
|
||
| import os | ||
|
|
||
| def main(): |
There was a problem hiding this comment.
[Flake8] expected 2 blank lines, found 1 (view)
| Rule |
|---|
E302 |
You can close this issue if no need to fix it. Learn more.
| print(os.environ['MSC_CMD_FLAGS']) | ||
| print(os.environ['MSC_IDE_FLAGS']) | ||
|
|
||
| if __name__ == "__main__": |
There was a problem hiding this comment.
[Flake8] expected 2 blank lines after class or function definition, found 1 (view)
| Rule |
|---|
E305 |
You can close this issue if no need to fix it. Learn more.
tools/VisualStudio/cl-dME.py
Outdated
| self.parser.add_argument( | ||
| '-D', | ||
| dest='defs' | ||
| action='append', |
There was a problem hiding this comment.
[Flake8] SyntaxError: invalid syntax (view)
| Rule |
|---|
E999 |
You can close this issue if no need to fix it. Learn more.
|
This pull request introduces 1 alert when merging 67e2e3e into c1a8460 - view on LGTM.com new alerts:
|
tools/VisualStudio/cl-dME.py
Outdated
|
|
||
| class DefinePrinter: | ||
| def __init__(): | ||
| self.parser = ArgumentParser(add_help=False) |
There was a problem hiding this comment.
[Flake8] undefined name 'self' (view)
| Rule |
|---|
F821 |
You can close this issue if no need to fix it. Learn more.
tools/VisualStudio/cl-dME.py
Outdated
| class DefinePrinter: | ||
| def __init__(): | ||
| self.parser = ArgumentParser(add_help=False) | ||
| self.parser.add_argument( |
There was a problem hiding this comment.
[Flake8] undefined name 'self' (view)
| Rule |
|---|
F821 |
You can close this issue if no need to fix it. Learn more.
|
This pull request introduces 1 alert when merging 291a992 into c1a8460 - view on LGTM.com new alerts:
|
tools/VisualStudio/cl-dME.py
Outdated
|
|
||
| def printto(self, key): | ||
| print(key) | ||
| if os.environ.has_key(key): |
There was a problem hiding this comment.
[Flake8] .has_key() is deprecated, use 'in' (view)
| Rule |
|---|
W601 |
You can close this issue if no need to fix it. Learn more.
tools/VisualStudio/cl-dME.py
Outdated
| # cl /B1cl-dME.bat main.cpp | ||
| # cl /BXcl-dME.bat main.cpp | ||
|
|
||
| import os |
There was a problem hiding this comment.
[Flake8] 'os' imported but unused (view)
| Rule |
|---|
F401 |
You can close this issue if no need to fix it. Learn more.
|
This pull request introduces 1 alert when merging 2483713 into c1a8460 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging 9847bbb into c1a8460 - view on LGTM.com new alerts:
|
|
This pull request introduces 1 alert when merging f2800d6 into f4124f9 - view on LGTM.com new alerts:
|
fixed #241