[FFmpeg-devel] [PATCH] vf_deshake: rename Transform.vector to Transform.vec to avoid compiler confusion

Michael Niedermayer michaelni at gmx.at
Thu Sep 18 00:59:33 CEST 2014


On Thu, Sep 18, 2014 at 12:16:42AM +0200, Andreas Cadhalpun wrote:
> Hi,
> 
> FFmpeg failed to build in Debian on powerpc/ppc64el with the
> following error:
> libavfilter/deshake.h:51:24: warning: declaration does not declare anything
>      MotionVector vector;  ///< Motion vector
>                         ^
> libavfilter/vf_deshake.c: In function 'find_motion':
> libavfilter/vf_deshake.c:297:20: error: expected identifier before
> '__attribute__'
>                  t->vector.x = x - deshake->rx;
>                     ^
> 
> Attached patch fixes this problem by renaming vector to vec.
> 
> Best regards,
> Andreas

>  deshake.h    |    2 +-
>  vf_deshake.c |   50 +++++++++++++++++++++++++-------------------------
>  2 files changed, 26 insertions(+), 26 deletions(-)
> fc1e8bc509d59d55b3cc68b37f78c749707195c0  vf_deshake-rename-Transform.vector-to-Transform.vec-.patch
> From 4288c94547fff0cbe6a3065f529bc8d732f61308 Mon Sep 17 00:00:00 2001
> From: Andreas Cadhalpun <Andreas.Cadhalpun at googlemail.com>
> Date: Thu, 18 Sep 2014 00:08:05 +0200
> Subject: [PATCH] vf_deshake: rename Transform.vector to Transform.vec to avoid
>  compiler confusion

applied

thanks

[...]

-- 
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/20140918/c485b33e/attachment.asc>


More information about the ffmpeg-devel mailing list