lkml.org 
[lkml]   [2012]   [Mar]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    SubjectRe: [RFC] Remove most all #define pr_fmt(fmt) lines
    From
    On Tue, Mar 27, 2012 at 19:23, Joe Perches <joe@perches.com> wrote:
    >  #ifndef pr_fmt
    > +
    > +#ifndef pr_fmt_std
    > +#define pr_fmt_std(fmt) KBUILD_MODNAME ": " fmt
    > +#endif
    > +#ifndef pr_fmt_dbg
    > +#define pr_fmt_dbg(fmt) fmt
    > +#endif
    >  #define pr_fmt(fmt) fmt

    What's the rationale behind the above line?
    I had expected

    #define pr_fmt(fmt) pr_fmt_std(fmt)

    > +
    > +#else
    > +
    > +#ifndef pr_fmt_std
    > +#define pr_fmt_std(fmt...) pr_fmt(fmt)

    Why the "fmt..." here, and "fmt" in the definitions above?

    > +#endif
    > +#ifndef pr_fmt_dbg
    > +#define pr_fmt_dbg(fmt...) pr_fmt(fmt)
    > +#endif
    > +
    >  #endif

    Gr{oetje,eeting}s,

                            Geert

    --
    Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

    In personal conversations with technical people, I call myself a hacker. But
    when I'm talking to journalists I just say "programmer" or something like that.
                                    -- Linus Torvalds
    --
    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: 2012-03-28 09:45    [W:0.025 / U:31.172 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site