From 8161b5c311d49f3a0313fe913d9ac24a9a6f9db6 Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Tue, 4 Aug 2020 14:15:15 +0200 Subject: [PATCH] Mark potentially-static methods as such Eclipse is finding a few methods which can be made static, make them static. Change-Id: I02776be45146e906f9c6a073204bf240cfe7c12f Signed-off-by: Robert Varga --- .../java/org/opendaylight/genius/idmanager/IdManager.java | 2 +- .../opendaylight/genius/idmanager/test/IdManagerTest.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/idmanager/idmanager-impl/src/main/java/org/opendaylight/genius/idmanager/IdManager.java b/idmanager/idmanager-impl/src/main/java/org/opendaylight/genius/idmanager/IdManager.java index 0019c0f86..85081acb4 100644 --- a/idmanager/idmanager-impl/src/main/java/org/opendaylight/genius/idmanager/IdManager.java +++ b/idmanager/idmanager-impl/src/main/java/org/opendaylight/genius/idmanager/IdManager.java @@ -535,7 +535,7 @@ public class IdManager implements IdManagerService, IdManagerMonitor { return 0; } - private long allocateIdBlockFromReleasedIdsHolder(IdLocalPool localIdPool, + private static long allocateIdBlockFromReleasedIdsHolder(IdLocalPool localIdPool, ReleasedIdsHolderBuilder releasedIdsBuilderParent, IdPool parentIdPool, TypedWriteTransaction confTx) { if (releasedIdsBuilderParent.getAvailableIdCount().toJava() == 0) { diff --git a/idmanager/idmanager-impl/src/test/java/org/opendaylight/genius/idmanager/test/IdManagerTest.java b/idmanager/idmanager-impl/src/test/java/org/opendaylight/genius/idmanager/test/IdManagerTest.java index ff81d9738..3dd4d6dda 100644 --- a/idmanager/idmanager-impl/src/test/java/org/opendaylight/genius/idmanager/test/IdManagerTest.java +++ b/idmanager/idmanager-impl/src/test/java/org/opendaylight/genius/idmanager/test/IdManagerTest.java @@ -365,13 +365,13 @@ public class IdManagerTest { } } - private InstanceIdentifier getIdPoolIdentifier(String poolName) { + private static InstanceIdentifier getIdPoolIdentifier(String poolName) { InstanceIdentifier.InstanceIdentifierBuilder idBuilder = InstanceIdentifier.builder(IdPools.class).child(IdPool.class, new IdPoolKey(poolName)); return idBuilder.build(); } - private ReleasedIdsHolder createReleaseIdHolder(List delayedIds) { + private static ReleasedIdsHolder createReleaseIdHolder(List delayedIds) { List delayedIdEntries = new ArrayList<>(); for (Long id : delayedIds) { delayedIdEntries.add(new DelayedIdEntriesBuilder().setId(id).setReadyTimeSec(0L).build()); @@ -380,7 +380,7 @@ public class IdManagerTest { .setAvailableIdCount((long) delayedIds.size()).build(); } - private AvailableIdsHolder createAvailableIdHolder(long start, long end, long cursor) { + private static AvailableIdsHolder createAvailableIdHolder(long start, long end, long cursor) { return new AvailableIdsHolderBuilder().setStart(start).setEnd(end).setCursor(cursor).build(); } -- 2.36.6