opendev/gerrit
Code Issues Proposed changes

Minor CheckStyle cleanups

Browse Source 
- Remove redundant 'final' modifier on private native method

- Fix line wrapping of opening brace

Change-Id: I5efcc7f2bdd8b7786f14319c23cb5c3a2c221591
This commit is contained in:
David Pursehouse
2016-09-22 17:38:49 +02:00
parent a620c89eeb
commit 9933f26e69
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-gwtui-common/src/main/java/com/google/gerrit/client/info/ChangeInfo.java
View File

@@ -460,7 +460,7 @@ public class ChangeInfo extends JavaScriptObject {
public final native String message() /*-{ return this.message; }-*/;
public final native int _revisionNumber() /*-{ return this._revision_number || 0; }-*/;
public final native String tag() /*-{ return this.tag; }-*/;
private final native String dateRaw() /*-{ return this.date; }-*/;
private native String dateRaw() /*-{ return this.date; }-*/;
public final Timestamp date() {
return JavaSqlTimestamp_JsonSerializer.parseTimestamp(dateRaw());

3
gerrit-server/src/main/java/com/google/gerrit/server/change/DeleteAssignee.java
View File

@@ -71,8 +71,7 @@ public class DeleteAssignee implements
@Override
public Response<AccountInfo> apply(ChangeResource rsrc, Input input)
throws RestApiException, UpdateException
{
throws RestApiException, UpdateException {
try (BatchUpdate bu = batchUpdateFactory.create(db.get(),
rsrc.getProject(),
rsrc.getUser(), TimeUtil.nowTs())) {