Skip to content

Commit 80f70be

Browse files
committed
Fix error handling for ConnectionStringConverter.
1 parent db87307 commit 80f70be

File tree

2 files changed

+5
-8
lines changed

2 files changed

+5
-8
lines changed

src/main/java/io/github/utplsql/cli/Cli.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,12 @@ public static void main(String[] args) {
2525
jc.usage();
2626
}
2727
} catch (ParameterException e) {
28-
if (jc.getParsedCommand() != null)
28+
if (jc.getParsedCommand() != null) {
29+
System.err.println(e.getMessage());
2930
jc.usage(jc.getParsedCommand());
30-
else
31+
} else {
3132
jc.usage();
33+
}
3234
} catch (Exception e) {
3335
e.printStackTrace();
3436
}
Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package io.github.utplsql.cli;
22

33
import com.beust.jcommander.IStringConverter;
4-
import com.beust.jcommander.ParameterException;
54

65
/**
76
* Created by Vinicius on 21/04/2017.
@@ -10,10 +9,6 @@ public class ConnectionStringConverter implements IStringConverter<ConnectionInf
109

1110
@Override
1211
public ConnectionInfo convert(String s) {
13-
try {
14-
return new ConnectionInfo().parseConnectionString(s);
15-
} catch (ParameterException ignored) {
16-
return null;
17-
}
12+
return new ConnectionInfo().parseConnectionString(s);
1813
}
1914
}

0 commit comments

Comments
 (0)