lkml.org 
[lkml]   [2006]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [patch 00/2] improve .text size on gcc 4.0 and newer compilers

    * Mitchell Blank Jr <mitch@sfgoth.com> wrote:

    > I actually did that in a project once (an "unlikely_if()" macro) It
    > was not a good idea. The problem is that every syntax-highlighter
    > knows that "if" is a keyword but you'd have to teach it about
    > "unlikely_if". It was surprising how visually jarring having
    > different pretty-printing for different types of "if" statements was.
    > "if (unlikely())" looks much cleaner in comparison.

    a better syntax would be:

    if __unlikely (cond) {
    ...
    }
    since it's the extra parantheses that are causing the visual complexity.

    Ingo
    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-01-06 02:04    [from the cache]
    ©2003-2011 Jasper Spaans. Advertise on this site