[FFmpeg-devel] [PATCH] libavformat/mxfdec: refactor reading strong ref array
Michael Niedermayer
michaelni at gmx.at
Wed Mar 4 12:19:41 CET 2015
On Sun, Mar 01, 2015 at 06:33:10PM -0800, Mark Reid wrote:
> hi,
> I was unsure whether this should be a function or a macro, I went with a function.
i think functions are better, especially when you try to read a
backtrace in gdb
then again i think macros are better when you wonder why gcc didnt
inline that speed critical and trivial fuction and made the whole
software run half speed
I think the first case applies more often though
>
> ---
> libavformat/mxfdec.c | 61 +++++++++++++++++++---------------------------------
> 1 file changed, 22 insertions(+), 39 deletions(-)
applied
thanks
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
There will always be a question for which you do not know the correct answer.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150304/15fa527a/attachment.asc>
More information about the ffmpeg-devel
mailing list