diff options
author | Uwe Stieber | 2012-05-25 09:17:14 +0000 |
---|---|---|
committer | Uwe Stieber | 2012-05-25 09:17:14 +0000 |
commit | c43ff2700cbc527f99e29eefe83f4586a4c2258e (patch) | |
tree | 483a85c908377cb05776c68fc06de57bb7f88e85 /target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse | |
parent | cd822fb9e1041dbd02071a04b2b7963ac98be248 (diff) | |
download | org.eclipse.tcf-c43ff2700cbc527f99e29eefe83f4586a4c2258e.tar.gz org.eclipse.tcf-c43ff2700cbc527f99e29eefe83f4586a4c2258e.tar.xz org.eclipse.tcf-c43ff2700cbc527f99e29eefe83f4586a4c2258e.zip |
Target Explorer: Fix more findbugs warnings
Diffstat (limited to 'target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse')
2 files changed, 13 insertions, 10 deletions
diff --git a/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/launcher/ScriptLauncher.java b/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/launcher/ScriptLauncher.java index a16b2ad43..3cbde1f2d 100644 --- a/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/launcher/ScriptLauncher.java +++ b/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/launcher/ScriptLauncher.java @@ -139,7 +139,9 @@ public class ScriptLauncher extends PlatformObject implements IScriptLauncher { */
@Override
public void onChannelClosed(Throwable error) {
- if (traceListener != null) { ((AbstractChannel)ScriptLauncher.this.channel).removeTraceListener(traceListener); traceListener = null; }
+ if (traceListener != null && ScriptLauncher.this.channel != null) {
+ ((AbstractChannel)ScriptLauncher.this.channel).removeTraceListener(traceListener); traceListener = null;
+ }
if (error != null) {
IStatus status = new Status(IStatus.ERROR, CoreBundleActivator.getUniqueIdentifier(),
NLS.bind(Messages.ScriptLauncher_error_channelConnectFailed, peer.getID(), error.getLocalizedMessage()),
@@ -320,7 +322,7 @@ public class ScriptLauncher extends PlatformObject implements IScriptLauncher { Token token = tokens[index];
- IService service = channel.getRemoteService(token.getServiceName());
+ IService service = channel != null ? channel.getRemoteService(token.getServiceName()) : null;
if (service != null) {
new Command(channel, service, token.getCommandName(), token.getArguments()) {
diff --git a/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/parser/Parser.java b/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/parser/Parser.java index 8d26de463..a3ab90eea 100644 --- a/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/parser/Parser.java +++ b/target_explorer/plugins/org.eclipse.tcf.te.tcf.core.scripting/src/org/eclipse/tcf/te/tcf/core/scripting/parser/Parser.java @@ -113,18 +113,19 @@ public class Parser { if (tok.startsWith("\"")) { //$NON-NLS-1$
// String type
- String fullTok = tok;
- boolean complete = isComplete(fullTok, '"', '"');
+ StringBuilder fullTok = new StringBuilder(tok);
+ boolean complete = isComplete(fullTok.toString(), '"', '"');
while (!complete && tokenizer.hasMoreTokens()) {
- fullTok = fullTok + " " + tokenizer.nextToken(); //$NON-NLS-1$
- complete = isComplete(fullTok, '"', '"');
+ fullTok.append(" "); //$NON-NLS-1$
+ fullTok.append(tokenizer.nextToken());
+ complete = isComplete(fullTok.toString(), '"', '"');
}
if (complete) {
- fullTok = fullTok.trim();
- if (fullTok.startsWith("\"")) fullTok = fullTok.substring(1); //$NON-NLS-1$
- if (fullTok.endsWith("\"")) fullTok = fullTok.substring(0, fullTok.length() - 1); //$NON-NLS-1$
- token.addArgument(fullTok);
+ String fullTokStr = fullTok.toString().trim();
+ if (fullTokStr.startsWith("\"")) fullTokStr = fullTokStr.substring(1); //$NON-NLS-1$
+ if (fullTokStr.endsWith("\"")) fullTokStr = fullTokStr.substring(0, fullTok.length() - 1); //$NON-NLS-1$
+ token.addArgument(fullTokStr);
continue;
}
}
|