On 09/03/2010 02:13 AM, Finn Thain wrote:
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
so the safest is to just ignore the column limit for urls to make things
legit.
Justin P. Mattock
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
