opendev/gerrit
Code Issues Proposed changes

Removed unnecessary Math.round() call

Browse Source 
Result of the operation is always an integer so the call to Math.round() 
is redundant.

Change-Id: I19fe01b0d71241012ad9f2b5f5c11fabf2b216f5
Signed-off-by: Eryk Szymanski <eryksz@gmail.com>
This commit is contained in:
Eryk Szymanski
2014-01-29 00:20:29 +01:00
parent 5579c284d5
commit 945fbd4bea
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-gwtui/src/main/java/com/google/gerrit/client/changes/ChangeTable2.java
View File

@@ -329,7 +329,7 @@ public class ChangeTable2 extends NavigationTable<ChangeInfo> {
int changedLines = c.insertions() + c.deletions(); int changedLines = c.insertions() + c.deletions();
int p = 100; int p = 100;
if (changedLines < largeChangeSize) { if (changedLines < largeChangeSize) {
p = Math.round(changedLines * 100 / largeChangeSize); p = changedLines * 100 / largeChangeSize;
} }
int width = Math.max(2, 70 * p / 100); int width = Math.max(2, 70 * p / 100);