diff options
| author |  laurentlb <laurentlb@google.com> | 2017-11-25 06:55:54 -0800 |
|---|---|---|
| committer |  Copybara-Service <copybara-piper@google.com> | 2017-11-25 06:58:06 -0800 |
| commit | a1c73f775cb8f56914d40ec603f8489eee5110eb (patch) | |
| tree | 6f49e63286e106571ddcaea3568d8d28019d6ddb /src/tools/skylark/java/com/google/devtools | |
| parent | 6fc836bbbca1c9c2473e3bf50cc4955f23f1a79e (diff) | |
Improve error message for 'missing-return-value' check.
RELNOTES: None.
PiperOrigin-RevId: 176899549
Diffstat (limited to 'src/tools/skylark/java/com/google/devtools')
| -rw-r--r-- | src/tools/skylark/java/com/google/devtools/skylark/skylint/ControlFlowChecker.java | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/tools/skylark/java/com/google/devtools/skylark/skylint/ControlFlowChecker.java b/src/tools/skylark/java/com/google/devtools/skylark/skylint/ControlFlowChecker.java index b7f94e9516..db28c52f6d 100644 --- a/src/tools/skylark/java/com/google/devtools/skylark/skylint/ControlFlowChecker.java +++ b/src/tools/skylark/java/com/google/devtools/skylark/skylint/ControlFlowChecker.java @@ -166,6 +166,7 @@ public class ControlFlowChecker extends SyntaxTreeVisitor { "some but not all execution paths of '" + node.getIdentifier() + "' return a value." + + " If it is intentional, make it explicit using 'return None'." + " If you know these cannot happen," + " add the statement `fail('unreachable')` to them." + " For more details, have a look at the documentation.", |