Retry if loading of a file fails

[rkta]

This fixes GH issue #210, Debian BTS #5377611 and obsoletes Debian
BTS #9464402.

File names like 'a#a.html', 'b?b.html' or 'c%20.html' can not be opened
without using '-o argv_is_url=1' as the file name is interpreted as a
local URL.

If everything fails and argv_is_url is not set retry as a local file.

[N-R-K]

Style nit: I'd put braces around it. Or maybe something like the following:

Suggested change

	if (getURLScheme(&url) == SCM_MISSING && !ArgvIsURL)
	retry_as_local_file:
	url = file_to_url(load_argv[i]);
	else
	retry_as_local_file:
	if (retry || (getURLScheme(&url) == SCM_MISSING && !ArgvIsURL))
	url = file_to_url(load_argv[i]);
	else

[rkta]

On Thu, May 26, 2022 at 03:40:49PM -0700, N-R-K wrote: @N-R-K commented on this pull request. > if (getURLScheme(&url) == SCM_MISSING && !ArgvIsURL) + retry_as_local_file: url = file_to_url(load_argv[i]); else Style nit: I'd put braces around it. Or maybe something like the following:

General style is to not use braces if a single statement will do. Having a label after the 'if' is a special case which is not present in the current code base. I'll follow the majority here.

```suggestion retry_as_local_file: if (retry || (getURLScheme(&url) == SCM_MISSING && !ArgvIsURL)) url = file_to_url(load_argv[i]); else ```

I don't like this suggestion. Checking the condition after jumping to the label is redundant. We'll only jump to the label after checking the condition.

[tats]

Merged, thanks for your contribution.