commit 56a5c275844f124bd01f13e7d4ed0a223977b878
author Annie Meng <anniemeng@google.com> Fri Nov 16 15:12:04 2018 +0000
committer Annie Meng <anniemeng@google.com> Fri Nov 16 17:49:16 2018 +0000
tree bf186029b589202dbf17a9b8908a334831722501
parent 92052f6f8e35886f92b65268e20112c28abac222

[Multi-user] Fix BMS checkstyle

Now that checkstyle runs a preupload hook for frameworks/services, when
we move BMS -> UserBMS, checkstyle will complain of errors because it
sees it as a new file.

This CL just fixes checkstyle complaints (mostly adding javadocs,
variable naming, operand placement, etc.) for BMS so the CL that creates
UserBMS can have as little changes as possible to ease review.

There is no change in functionality in this CL.

Bug: 118520567
Test: atest RunFrameworksServicesRoboTests
Change-Id: I1118d6deef83d1abfbc8fd94883d7d162a4399cb