lkml.org 
[lkml]   [2008]   [Apr]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/2] ocfs2: replace remaining __PRETTY_FUNCTION__ occurrences
On Fri, Apr 04, 2008 at 02:28:00PM -0700, Harvey Harrison wrote:
> The kernel is written in C, not C++, use __func__

__PRETTY_FUNCTION__ is GNU C, but __func__ is still better.
Acked-by: Joel Becker <joel.becker@oracle.com>

>
> Signed-off-by: Harvey Harrison <harvey.harrison@gmail.com>
> ---
> fs/ocfs2/cluster/masklog.h | 2 +-
> fs/ocfs2/dlmglue.c | 2 +-
> fs/ocfs2/super.h | 4 ++--
> 3 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/fs/ocfs2/cluster/masklog.h b/fs/ocfs2/cluster/masklog.h
> index 597e064..8bdb182 100644
> --- a/fs/ocfs2/cluster/masklog.h
> +++ b/fs/ocfs2/cluster/masklog.h
> @@ -193,7 +193,7 @@ extern struct mlog_bits mlog_and_bits, mlog_not_bits;
> */
> #define __mlog_printk(level, fmt, args...) \
> printk(level "(%u,%lu):%s:%d " fmt, task_pid_nr(current), \
> - __mlog_cpu_guess, __PRETTY_FUNCTION__, __LINE__ , \
> + __mlog_cpu_guess, __func__, __LINE__ , \
> ##args)
>
> #define mlog(mask, fmt, args...) do { \
> diff --git a/fs/ocfs2/dlmglue.c b/fs/ocfs2/dlmglue.c
> index 1f1873b..5a3f1f4 100644
> --- a/fs/ocfs2/dlmglue.c
> +++ b/fs/ocfs2/dlmglue.c
> @@ -105,7 +105,7 @@ static void ocfs2_dentry_post_unlock(struct ocfs2_super *osb,
> struct ocfs2_lock_res *lockres);
>
>
> -#define mlog_meta_lvb(__level, __lockres) ocfs2_dump_meta_lvb_info(__level, __PRETTY_FUNCTION__, __LINE__, __lockres)
> +#define mlog_meta_lvb(__level, __lockres) ocfs2_dump_meta_lvb_info(__level, __func__, __LINE__, __lockres)
>
> /* This aids in debugging situations where a bad LVB might be involved. */
> static void ocfs2_dump_meta_lvb_info(u64 level,
> diff --git a/fs/ocfs2/super.h b/fs/ocfs2/super.h
> index 783f527..15e2d98 100644
> --- a/fs/ocfs2/super.h
> +++ b/fs/ocfs2/super.h
> @@ -36,13 +36,13 @@ void __ocfs2_error(struct super_block *sb,
> const char *fmt, ...)
> __attribute__ ((format (printf, 3, 4)));
>
> -#define ocfs2_error(sb, fmt, args...) __ocfs2_error(sb, __PRETTY_FUNCTION__, fmt, ##args)
> +#define ocfs2_error(sb, fmt, args...) __ocfs2_error(sb, __func__, fmt, ##args)
>
> void __ocfs2_abort(struct super_block *sb,
> const char *function,
> const char *fmt, ...)
> __attribute__ ((format (printf, 3, 4)));
>
> -#define ocfs2_abort(sb, fmt, args...) __ocfs2_abort(sb, __PRETTY_FUNCTION__, fmt, ##args)
> +#define ocfs2_abort(sb, fmt, args...) __ocfs2_abort(sb, __func__, fmt, ##args)
>
> #endif /* OCFS2_SUPER_H */
> --
> 1.5.5.rc1.135.g8527
>
>
> --
> 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/
>

--

"I'm living so far beyond my income that we may almost be said
to be living apart."
- e e cummings

Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker@oracle.com
Phone: (650) 506-8127


\
 
 \ /
  Last update: 2008-04-04 23:39    [W:0.032 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site