Raincloud/sheepit-shadow-nabber
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Bugfix: merge the configuration and settings (Configuration i.e. command line args have high priority)

Browse Source
This commit is contained in:
Laurent Clouet
2015-03-31 23:06:21 +01:00
parent ad6547d3d9
commit 901fab65f2
2 changed files with 10 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
src/com/sheepit/client/standalone/swing/SettingsLoader.java
View File

@@ -144,6 +144,10 @@ public class SettingsLoader {
}
}
/**
* Merge the Settings file with the Configuration.
* The Configuration will have high priority.
*/
public void merge(Configuration config) {
if (config == null) {
System.out.println("SettingsLoader::merge config is null");
@@ -151,23 +155,23 @@ public class SettingsLoader {
loadFile();
if (login != null) {
if (config.login().isEmpty() && login != null) {
config.setLogin(login);
}
if (password != null) {
if (config.password().isEmpty() && password != null) {
config.setPassword(password);
}
if (computeMethod != null) {
if (config.getComputeMethod() == null && computeMethod != null) {
config.setComputeMethod(ComputeType.valueOf(computeMethod));
}
if (gpu != null) {
if (config.getGPUDevice() == null && gpu != null) {
GPUDevice device = GPU.getGPUDevice(gpu);
if (device != null) {
config.setUseGPU(device);
}
}
if (cacheDir != null) {
if (config.getUserSpecifiedACacheDir() == false && cacheDir != null) {
config.setCacheDir(new File(cacheDir));
}
}