mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-07-04 23:55:26 +02:00
Code Issues Wiki Activity

[401476] Strip whitespace around Terminal Telnet Port

Browse source 
Change-Id: I45857606c543a6d41037f5bcb67158c3dba92f38
Signed-off-by: Martin Oberhuber <martin.oberhuber@windriver.com>
This commit is contained in:
Martin Oberhuber 2013-02-21 22:55:58 +01:00
parent 37aa12df91
commit eafd89e229
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
plugins/org.eclipse.tm.terminal.telnet/src/org/eclipse/tm/internal/terminal/telnet/TelnetSettingsPage.java
View file

@ -14,6 +14,7 @@
* Michael Scharf (Wind River) - extracted from TerminalSettingsDlg
* Martin Oberhuber (Wind River) - fixed copyright headers and beautified
* Martin Oberhuber (Wind River) - [206917] Add validation for Terminal Settings
* Martin Oberhuber (Wind River) - [401476] Strip whitespace around Telnet Port
*******************************************************************************/
package org.eclipse.tm.internal.terminal.telnet;
@ -86,7 +87,7 @@ public class TelnetSettingsPage extends AbstractSettingsPage {
}
}
private String getNetworkPort() {
String portText = fNetworkPortCombo.getText();
String portText = fNetworkPortCombo.getText().trim();
String mappedPort = getNetworkPortMap().findPort(portText);
return mappedPort!=null ? mappedPort : portText;
}
@ -111,7 +112,7 @@ public class TelnetSettingsPage extends AbstractSettingsPage {
}
try {
int p = Integer.parseInt(getNetworkPort().trim());
int p = Integer.parseInt(getNetworkPort());
if (p <= 0 || p > 65535) {
String m = "Invalid network port. Must be between 0 and 65535."; //$NON-NLS-1$
int mt = IMessageProvider.ERROR;