Juan Quintela <quint...@redhat.com> writes: > Alistair Francis <alistair.fran...@xilinx.com> wrote: >> Replace all occurs of __FUNCTION__ except for the check in checkpatch >> with the non GCC specific __func__. >> >> One line in hcd-musb.c was manually tweaked to pass checkpatch. >> >> Signed-off-by: Alistair Francis <alistair.fran...@xilinx.com> >> Cc: Gerd Hoffmann <kra...@redhat.com> >> Cc: Andrzej Zaborowski <balr...@gmail.com> >> Cc: Stefano Stabellini <sstabell...@kernel.org> >> Cc: Anthony Perard <anthony.per...@citrix.com> >> Cc: John Snow <js...@redhat.com> >> Cc: Aurelien Jarno <aurel...@aurel32.net> >> Cc: Yongbok Kim <yongbok....@imgtec.com> >> Cc: Peter Crosthwaite <crosthwaite.pe...@gmail.com> >> Cc: Stefan Hajnoczi <stefa...@redhat.com> >> Cc: Fam Zheng <f...@redhat.com> >> Cc: Juan Quintela <quint...@redhat.com> >> Cc: "Dr. David Alan Gilbert" <dgilb...@redhat.com> >> Cc: qemu-...@nongnu.org >> Cc: qemu-block@nongnu.org >> Cc: xen-de...@lists.xenproject.org >> Reviewed-by: Eric Blake <ebl...@redhat.com> >> Reviewed-by: Stefan Hajnoczi <stefa...@redhat.com> > > Reviewed-by: Juan Quintela <quint...@redhat.com> > > >> diff --git a/audio/audio_int.h b/audio/audio_int.h >> index 5bcb1c60e1..543b1bd8d5 100644 >> --- a/audio/audio_int.h >> +++ b/audio/audio_int.h >> @@ -253,7 +253,7 @@ static inline int audio_ring_dist (int dst, int src, int >> len) >> #define AUDIO_STRINGIFY(n) AUDIO_STRINGIFY_(n) >> >> #if defined _MSC_VER || defined __GNUC__ >> -#define AUDIO_FUNC __FUNCTION__ >> +#define AUDIO_FUNC __func__ >> #else >> #define AUDIO_FUNC __FILE__ ":" AUDIO_STRINGIFY (__LINE__) >> #endif > > Unrelated to this patch .... > Do we really support other compilers than msc and gcc?
Let me rephrase the question: do we really support compilers that don't understand __func__? The presence of numerous unconditional uses of __func__ in the tree means the answer is no. Let's replace AUDIO_FUNC by plain __func__.