On Sat, 2017-11-18 at 11:07 +0100, Andreas Brauchli wrote: > Is the patch acceptable if it only catches URL with protocol specifiers > foo:// ? That would catch most cases and doesn't show false positives: > $ git grep -E '://\S{75}.*' -- '*.[ch]'
Of course. but I suggest using a prefix like \b[\w\.\+\-]+:// so the URL scheme is properly described. It might be best if there was some generic mechanism to identify possible URIs and not just URLs instead. https://en.wikipedia.org/wiki/Uniform_Resource_Identifier