[FFmpeg-devel] [PATCH 2/6] x86: hevc_mc: remove unneeded shift

Michael Niedermayer michaelni at gmx.at
Sun Jun 1 23:47:57 CEST 2014

On Sun, Jun 01, 2014 at 02:12:58PM +0000, Christophe Gisquet wrote:
> The immediate value may be 0.
> ---
>  libavcodec/x86/hevc_mc.asm | 10 ++++++++++
>  1 file changed, 10 insertions(+)


maybe this could be simplified by using a macro which evaluates to
nothing when its 2nd argument is 0


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Its not that you shouldnt use gotos but rather that you should write
readable code and code with gotos often but not always is less readable
-------------- 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/20140601/22d25d15/attachment.asc>

More information about the ffmpeg-devel mailing list