Fix minor CheckStyle warnings

- Correct modifier order to follow JLS recommendation
- Wrap closing braces

Change-Id: Iad5d4f9dd29b9d52a76c4f3be8776e75950b5346
This commit is contained in:
David Pursehouse
2016-08-29 14:15:49 +09:00
parent 47180f89d0
commit 69d35942c8
3 changed files with 5 additions and 3 deletions

View File

@@ -48,7 +48,8 @@ public class MyAgreementsScreen extends SettingsScreen {
@Override @Override
public void preDisplay(JsArray<AgreementInfo> result) { public void preDisplay(JsArray<AgreementInfo> result) {
agreements.display(Natives.asList(result)); agreements.display(Natives.asList(result));
}}); }
});
} }
private static class AgreementTable extends FancyFlexTable<ContributorAgreement> { private static class AgreementTable extends FancyFlexTable<ContributorAgreement> {

View File

@@ -85,7 +85,8 @@ public class NewAgreementScreen extends AccountScreen {
} }
postRPC(); postRPC();
} }
}}); }
});
available = Gerrit.info().auth().contributorAgreements(); available = Gerrit.info().auth().contributorAgreements();
postRPC(); postRPC();

View File

@@ -98,7 +98,7 @@ class ChangeNotesParser {
new RevId("INVALID PARTIAL PATCH SET"); new RevId("INVALID PARTIAL PATCH SET");
@AutoValue @AutoValue
static abstract class ApprovalKey { abstract static class ApprovalKey {
abstract PatchSet.Id psId(); abstract PatchSet.Id psId();
abstract Account.Id accountId(); abstract Account.Id accountId();
abstract String label(); abstract String label();