diff options
author | Taras Tsugrii <ttsugrii@fb.com> | 2018-02-13 09:12:12 -0800 |
---|---|---|
committer | Copybara-Service <copybara-piper@google.com> | 2018-02-13 09:14:16 -0800 |
commit | a3339c8ca2e824757afe698f591770f4232af530 (patch) | |
tree | 89c3494b7b8ec2156c8be4144adfd70cbf1333c5 /src/main/java/com/google/devtools/build/lib/runtime/FancyTerminalEventHandler.java | |
parent | 391d73d15f19ed22bf2cb343aaca9dd4fe3b7285 (diff) |
[Skylint] Do not crash ControlFlowChecker on nested functions.
ControlFlowChecker has a precondition check that assumes nested functions do not occur. While this assumption is reasonable for a valid Skylark syntax, linter can actually be invoked on malformed files and users would get a stack trace instead of a human-readable linter error.
Alternative and possibly a better strategy would to not run CFChecker in case parse errors are detected.
fixes #4511
Closes #4512.
PiperOrigin-RevId: 185538897
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/runtime/FancyTerminalEventHandler.java')
0 files changed, 0 insertions, 0 deletions