Re: [PATCH] checkpatch: allow URL >80 chars
From: Joe Perches
Date: Sat Nov 18 2017 - 09:40:15 EST
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