From 8da179f477b9bb95f9fa470fcae421b0b9b94018 Mon Sep 17 00:00:00 2001
From: David Bohn <davbohn@googlemail.com>
Date: Thu, 23 Jun 2016 00:30:45 +0200
Subject: [PATCH] Some minor changes to make the IDE happy
---
.../configurations/AbstractConfiguration.java | 14 ++++++--------
.../configurations/DefaultConfiguration.java | 1 -
2 files changed, 6 insertions(+), 9 deletions(-)
diff --git a/src/main/java/fucoin/configurations/AbstractConfiguration.java b/src/main/java/fucoin/configurations/AbstractConfiguration.java
index 21902b6..3693fdd 100644
--- a/src/main/java/fucoin/configurations/AbstractConfiguration.java
+++ b/src/main/java/fucoin/configurations/AbstractConfiguration.java
@@ -23,11 +23,11 @@ import java.util.concurrent.ThreadLocalRandom;
*/
public abstract class AbstractConfiguration extends AbstractNode {
- protected ActorRef superVisor;
+ private ActorRef superVisor;
- protected final List<ActorRef> activeActors = new ArrayList<>();
+ private final List<ActorRef> activeActors = new ArrayList<>();
- protected Timeout timeout = new Timeout(Duration.create(10, "seconds"));
+ private Timeout timeout = new Timeout(Duration.create(10, "seconds"));
public static Props props(Class configurationClass) {
@@ -38,7 +38,7 @@ public abstract class AbstractConfiguration extends AbstractNode {
* Spawns a new wallet and blocks until it has received its initial money
* @throws Exception on timeout
*/
- public ActorRef spawnWallet(String name, boolean createGUI) throws Exception {
+ ActorRef spawnWallet(String name, boolean createGUI) throws Exception {
Future<Object> future = Patterns.ask(superVisor, new ActionAnnounceWalletCreation(1, self()), timeout);
ActorRef wallet = createWallet(name, createGUI);
Await.result(future, timeout.duration());
@@ -48,7 +48,7 @@ public abstract class AbstractConfiguration extends AbstractNode {
/**
* Creates a wallet without blocking until the wallet was created
*/
- public ActorRef createWallet(String name, boolean createGUI) {
+ private ActorRef createWallet(String name, boolean createGUI) {
Props props;
int numOfWallets = activeActors.size();
if (numOfWallets == 0) {
@@ -83,7 +83,6 @@ public abstract class AbstractConfiguration extends AbstractNode {
/**
* Fetch a random wallet
- * @return
*/
public ActorRef getRandomWallet() {
return activeActors.get(ThreadLocalRandom.current().nextInt(activeActors.size()));
@@ -95,9 +94,8 @@ public abstract class AbstractConfiguration extends AbstractNode {
/**
* Create the supervisor node
- * @return
*/
- public ActorRef initSupervisor() {
+ ActorRef initSupervisor() {
superVisor = context().actorOf(SuperVisorImpl.props(), "SuperVisorImpl");
// Don't ask.
diff --git a/src/main/java/fucoin/configurations/DefaultConfiguration.java b/src/main/java/fucoin/configurations/DefaultConfiguration.java
index 555e6ef..84bbb59 100644
--- a/src/main/java/fucoin/configurations/DefaultConfiguration.java
+++ b/src/main/java/fucoin/configurations/DefaultConfiguration.java
@@ -1,7 +1,6 @@
package fucoin.configurations;
import akka.actor.ActorRef;
-import fucoin.actions.Action;
import fucoin.actions.control.ActionWalletSendMoney;
import fucoin.actions.transaction.ActionNotifyObserver;
import fucoin.configurations.internal.ConfigurationName;
--
GitLab