mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-06-08 18:26:01 +02:00
Code Issues Wiki Activity

Merge "Bug 461324 - Fix NPE in Remote Connections preference page"

Browse source
This commit is contained in:
Greg Watson 2015-03-05 17:27:15 -05:00 • committed by Gerrit Code Review @ Eclipse.org
commit fab5645c94
1 changed files with 26 additions and 27 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
bundles/org.eclipse.remote.core/src/org/eclipse/remote/internal/core/RemoteConnectionWorkingCopy.java
View file

@ -33,7 +33,7 @@ import org.osgi.service.prefs.Preferences;
public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy {
private RemoteConnection original;
private RemoteConnectionType connectionType;
private final RemoteConnectionType connectionType;
private String newName;
private final Map<String, String> newAttributes = new HashMap<>();
private final Map<String, String> newSecureAttributes = new HashMap<>();
@ -52,6 +52,7 @@ public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy
*/
public RemoteConnectionWorkingCopy(RemoteConnection original) {
this.original = original;
this.connectionType = (RemoteConnectionType) original.getConnectionType();
}
/*
@ -63,11 +64,13 @@ public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy
public String getName() {
if (newName != null) {
return newName;
} else if (original != null) {
return original.getName();
} else {
return null;
}
if (original != null) {
return original.getName();
}
return null;
}
/*
@ -77,7 +80,7 @@ public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy
*/
@Override
public void setName(String name) {
// set if only if it's changed
// set it only if it's changed
if (original == null || !name.equals(original.getName())) {
newName = name;
}
@ -224,12 +227,8 @@ public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy
*/
@Override
public <T extends Service> T getService(Class<T> service) {
if (original != null) {
return original.getService(service);
} else {
return connectionType.getConnectionService(this, service);
}
}
/*
* (non-Javadoc)
@ -373,16 +372,16 @@ public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy
public boolean isOpen() {
if (original != null) {
return original.isOpen();
} else {
}
IRemoteConnectionControlService controlService = connectionType.getConnectionService(this,
IRemoteConnectionControlService.class);
if (controlService != null) {
return controlService.isOpen();
} else {
}
return true;
}
}
}
/*
* (non-Javadoc)
@ -393,15 +392,15 @@ public class RemoteConnectionWorkingCopy implements IRemoteConnectionWorkingCopy
public String getProperty(String key) {
if (original != null) {
return original.getProperty(key);
} else {
}
IRemoteConnectionPropertyService propertyService = connectionType.getConnectionService(this,
IRemoteConnectionPropertyService.class);
if (propertyService != null) {
return propertyService.getProperty(key);
} else {
}
return null;
}
}
}
}