On Mon, 2017-11-20 at 13:40 +0100, Andreas Brauchli wrote: > Allow URL to exceed the 80 char limit for improved interaction in > adaption to ongoing but undocumented practice. > > $ git grep -E '://\S{77}.*' -- '*.[ch]' > > The patch checks that the URL is indeed on its own line in that it > allows a maximal prefix of 3 characters to account for a URL after a > comment (e.g. '// https://...')
I think this part is not necessary or that useful. For instance, this style /* some long url */ is pretty common. > As per RFC3986 [1], the URL format allows for alphanum, +, - and . > characters in the scheme before the separator :// as long as it starts > with a letter (e.g. https, git, f.-+). That seems right, thanks. > Recognition of URIs without more context information is prone to false > positives and thus currently left out of the heuristics. OK.