mirrors/cdt
1
0
Fork 0
mirror of https://github.com/eclipse-cdt/cdt synced 2025-08-01 21:35:40 +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 2e79b7eda4
commit fa6c90b7ab
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

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