Re: [FFmpeg-devel] [PATCH 5/8] avcodec/amr: replace #define by typedef

2015-11-21 Thread Ganesh Ajjanagadde
On Fri, Nov 20, 2015 at 3:01 PM, Michael Niedermayer wrote: > On Thu, Nov 19, 2015 at 08:14:12AM -0500, Ganesh Ajjanagadde wrote: >> See e.g >> https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c >> for rationale. >> >> Signed-off-by: Ganesh Ajjanagadde >> --- > > sh

Re: [FFmpeg-devel] [PATCH 5/8] avcodec/amr: replace #define by typedef

2015-11-20 Thread Michael Niedermayer
On Thu, Nov 19, 2015 at 08:14:12AM -0500, Ganesh Ajjanagadde wrote: > See e.g > https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c > for rationale. > > Signed-off-by: Ganesh Ajjanagadde > --- should be ok [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730B

[FFmpeg-devel] [PATCH 5/8] avcodec/amr: replace #define by typedef

2015-11-19 Thread Ganesh Ajjanagadde
See e.g https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c for rationale. Signed-off-by: Ganesh Ajjanagadde --- libavcodec/amr.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/amr.h b/libavcodec/amr.h index 1ac73ab..727f8c3 10064