Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 #!/usr/bin/python | |
| 2 | |
| 3 import subprocess | |
| 4 import sys | |
| 5 | |
| 6 def __main__(): | |
|
Eugene But (OOO till 7-30)
2015/10/08 16:10:41
Optional NIT: s/__main__/main ? Otherwise the name
Eugene But (OOO till 7-30)
2015/10/08 16:10:41
Need 2 blank lines before top level def
Dirk Pranke
2015/10/08 17:49:45
I wouldn't call this change optional. Please chang
Eugene But (OOO till 7-30)
2015/10/08 18:04:11
I use gpylint for this.
Dirk Pranke
2015/10/08 18:22:29
I meant, don't we have some other script that call
sdefresne
2015/10/09 13:54:45
I've looked around but could find nothing. There i
| |
| 7 return subprocess.check_call(["java", "-jar"] + sys.argv[1:]) | |
| 8 | |
| 9 if __name__ == "__main__": | |
| 10 __main__() | |
| OLD | NEW |