opendev/gerrit
Code Issues Proposed changes

Get rid of empty if-statements

Browse Source 
Change-Id: Ie1a6a8e60bcb7affd4c8fe7432d6bb93ca499637
This commit is contained in:
David Pursehouse
2015-03-16 17:43:54 +09:00
parent d70fc02a99
commit 9319d0de88
5 changed files with 8 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-server/src/main/java/com/google/gerrit/server/git/ReceiveCommitsAdvertiseRefsHook.java
View File

@@ -154,9 +154,10 @@ public class ReceiveCommitsAdvertiseRefsHook implements AdvertiseRefsHook {
RevCommit c;
try {
while ((c = rw.next()) != null && toInclude.size() < max) {
if (alreadySending.contains(c)) {
} else if (toInclude.contains(c)) {
} else if (c.getParentCount() > 1) {
if (alreadySending.contains(c)
|| toInclude.contains(c)
|| c.getParentCount() > 1) {
// Do nothing
} else if (toInclude.size() < base) {
toInclude.add(c);
} else {