diff options
author | Laurent Le Brun <laurentlb@google.com> | 2015-09-03 13:59:44 +0000 |
---|---|---|
committer | Damien Martin-Guillerez <dmarting@google.com> | 2015-09-03 14:42:33 +0000 |
commit | 54733f93471af0cbe63e9f32a522971dbd193a3e (patch) | |
tree | d7ae030fd727d129f80dd137cf6df3f5ba4fba41 /src/main/java/com/google/devtools/build/lib/runtime/commands | |
parent | 31d6bba4ec2041926063d26c7b53befcb647e8d5 (diff) |
Code cleanup
--
MOS_MIGRATED_REVID=102239051
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/runtime/commands')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/runtime/commands/HelpCommand.java | 2 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/runtime/commands/InfoCommand.java | 16 |
2 files changed, 8 insertions, 10 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/runtime/commands/HelpCommand.java b/src/main/java/com/google/devtools/build/lib/runtime/commands/HelpCommand.java index e168462ad2..4418f1eb8d 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/commands/HelpCommand.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/commands/HelpCommand.java @@ -226,7 +226,7 @@ public final class HelpCommand implements BlazeCommand { for (String name : commands) { BlazeCommand command = commandsByName.get(name); - String varName = name.toUpperCase().replace("-", "_"); + String varName = name.toUpperCase().replace('-', '_'); Command annotation = command.getClass().getAnnotation(Command.class); if (!annotation.completion().isEmpty()) { outErr.printOutLn("BAZEL_COMMAND_" + varName + "_ARGUMENT=\"" diff --git a/src/main/java/com/google/devtools/build/lib/runtime/commands/InfoCommand.java b/src/main/java/com/google/devtools/build/lib/runtime/commands/InfoCommand.java index d1f18cefcd..1a0566b9cd 100644 --- a/src/main/java/com/google/devtools/build/lib/runtime/commands/InfoCommand.java +++ b/src/main/java/com/google/devtools/build/lib/runtime/commands/InfoCommand.java @@ -365,15 +365,13 @@ public class InfoCommand implements BlazeCommand { AllowedRuleClassInfo.Builder info = AllowedRuleClassInfo.newBuilder(); info.setPolicy(AllowedRuleClassInfo.AllowedRuleClasses.ANY); - if (attr.isStrictLabelCheckingEnabled()) { - if (attr.getAllowedRuleClassesPredicate() != Predicates.<RuleClass>alwaysTrue()) { - info.setPolicy(AllowedRuleClassInfo.AllowedRuleClasses.SPECIFIED); - Predicate<RuleClass> filter = attr.getAllowedRuleClassesPredicate(); - for (RuleClass otherClass : Iterables.filter( - ruleClasses, filter)) { - if (otherClass.isDocumented()) { - info.addAllowedRuleClass(otherClass.getName()); - } + if (attr.isStrictLabelCheckingEnabled() + && attr.getAllowedRuleClassesPredicate() != Predicates.<RuleClass>alwaysTrue()) { + info.setPolicy(AllowedRuleClassInfo.AllowedRuleClasses.SPECIFIED); + Predicate<RuleClass> filter = attr.getAllowedRuleClassesPredicate(); + for (RuleClass otherClass : Iterables.filter(ruleClasses, filter)) { + if (otherClass.isDocumented()) { + info.addAllowedRuleClass(otherClass.getName()); } } } |