Add javadoc comment and change catched exception type

Author: travkin79

org.eclipse.lsp4e/src/org/eclipse/lsp4e/LSPEclipseUtils.java

@@ -1602,6 +1602,16 @@ public static Set<IEditorReference> findOpenEditorsFor(@Nullable URI uri) {
 		return null;
 	}
 
+	/**
+	 * Converts the given string to a {@link URI} if possible.
+	 * Ensures correct encoding in case of file URIs.
+	 *
+	 * @param uri a URI string
+	 * @return a URI instance for the given string
+	 * @throws IllegalArgumentException if the given string is not a valid URI (if it violates RFC 2396)
+	 * @throws NullPointerException if the given string is null
+	 * @see URI#create(String)
+	 */
 	public static URI toUri(String uri) {
 		URI initialUri = URI.create(uri);
 		return FILE_SCHEME.equals(initialUri.getScheme()) ? toUri(Path.fromPortableString(initialUri.getPath())) : initialUri;

org.eclipse.lsp4e/src/org/eclipse/lsp4e/operations/symbols/internal/SymbolIconProviderRegistry.java

@@ -144,7 +144,7 @@ private SymbolIconProvider getIconProvider(Object symbol) {
 
 		try {
 			return LSPEclipseUtils.toUri(uri);
-		} catch (Exception e) {
+		} catch (IllegalArgumentException | NullPointerException e) {
 			LanguageServerPlugin.logWarning("Failed to parse URI " + uri, e); //$NON-NLS-1$
 			return null;
 		}