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 [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
