[Github-comments] [geany/geany] Towards gvfs-fuse independence (#963)

[elextr]

elextr commented on this pull request.



> @@ -1788,6 +1788,9 @@ void utils_tidy_path(gchar *filename)
 	const gchar *needle;
 	gboolean preserve_double_backslash = FALSE;
 
+	if (utils_is_uri(filename))
+		return;

@techee I was almost going to say, its ok, for URLs from schemes that are likely to be used as files the `//` is present, but on checking I found that RFC 8089 was released this month for `file:` where one difference from the previous is "The syntax given in Section 2 makes the entire authority  component, including the double slashes "//", optional.".  So I would say the rule is now starts with `file:` or contains `://`.

Yes its a less likely URL, but people dragging URLs onto Geany might get one.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/963#discussion_r103344838
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20170227/6e0f4a8c/attachment.html>