[PATCH stable] Separate multicast configuration for mesh and wlan interfaces.
dwmw2 at infradead.org
Wed May 14 04:14:59 EDT 2008
On Tue, 2008-05-13 at 16:15 -0700, Andrew Morton wrote:
> On Tue, 13 May 2008 19:12:27 -0400 Andres Salomon <dilinger at queued.net> wrote:
> > And FWIW, I like the 80 char limit _except_ when it comes to strings.
> I don't normally bother about the strings, unless it is obvious that
> the surrounding code has worked to prevent them from wrapping (and if I
> notice that).
> Or if they make code particularly hard to read and alter. I've seen
> some which wander out to column 130, which is getting daft.
> The code at http://userweb.kernel.org/~akpm/x.jpg has short strings,
> but it has gone and stuffed the _arguments_ onto the same line too, which
> is just obnoxious.
I would probably accept patches to move the arguments onto the next
line; I don't think that would make the overall code less readable.
In fact, I would normally put the arguments on the next line these days
anyway -- but that particular piece of debugging code dates from before
all this pointless fuss about 80 columns got started, so it was never an
It could probably do with printk priorities too, while we're at it.
More information about the Devel