On Fri, 3 Sep 2010, Justin P. Mattock wrote:
On 09/03/2010 01:47 AM, Finn Thain wrote:
On Fri, 3 Sep 2010, Justin P. Mattock wrote:
On 09/03/2010 01:23 AM, Finn Thain wrote:
URLs can't be split across lines usefully, so disregard the
checkpatch complaint.
Finn
with the coding style, what files should ignore coding style with
the tab rule i.e. Kconfig *.c, *.h, README, etc... (from here
Kconfig make sense due to the ncurses interface..)
I'm not sure what you mean (?)
Kconfig seems to use the<...> syntax for URLs. checkpatch doesn't
complain about the ~90 character long URL in ./drivers/scsi/Kconfig...
Finn
yeah this is a bit confusing, especially with checkpatch in the mix.. So
if I understand correctly, using checkpatch, any address that uses the
column limit: (example)
+ http://web.archive.org/web/*/http://www.unix-ag.org/Linux-Alpha/
+ Architectures/Avanti.html.
breaks checkpatch(without the <.....>) in any of the files? as for
Kconfig make the url long so the she looks pretty for the user..
I don't know whether checkpatch special-cases Kconfig or angle brackets.
You'd have to put it to the test. But it is not important.
The point is, you can't line-wrap URLs without breaking software that
looks for them (text editors, browsers, terminal emulators ... your own
scripts too probably).
Finn
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
