[lkml]   [2009]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [GIT PULL] percpu fixes for 2.6.32-rc6

    11/13/2009 12:17 AM, Linus Torvalds wrote:
    > On Thu, 12 Nov 2009, Tejun Heo wrote:
    >> 11/12/2009 07:36 PM, Ingo Molnar wrote:
    >>> Breaking strings mid-sentence is something we try not to do. (If you
    >>> know about places that do it 'quite often' then those places need fixing
    >>> too.)
    >> Oh... I do that all the time and I see a lot of them around too.
    > I hate them. I do greps for error messages, and it's annoying as hell if
    > it's hard to find.
    > 'checkpatch' is the major reason for them, but I think we've fixed
    > checkpath long ago to not warn about long lines if they are due to a long
    > string.
    > Strings should basically be broken up only at '\n' characters, so
    > printk("This is a made-up example.\n"
    > "Ok like this\n");
    > is fine, because you can expect to grep for "made-up example", but not
    > over the newline.

    If the consensus is to allow long string literals to overrun 80 column
    limit, I have no objection. It makes code slightly more difficult to
    read for some people but well we can't make everyone happy.



     \ /
      Last update: 2009-11-12 16:35    [W:0.023 / U:16.472 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site