Nick Cronin wrote:
unlike http:// etc, file:/// has 3 slashes, this patch removes the third slash.
Could you provide a test case for this please?
Specifically, is "http:///foo.bar" different to "http://foo.bar%22? Your code treats them equally. How about "file:///" and "file://"?
Mike