Re: [FFmpeg-devel] [PATCH 07/15] vp8/x86: move variable assigned inside macro branch.

2014-12-27 Thread Ronald S. Bultje
Hi, On Sat, Dec 27, 2014 at 12:55 PM, Clément Bœsch wrote: > On Sat, Dec 27, 2014 at 11:02:42AM -0500, Ronald S. Bultje wrote: > > The value is not used outside the branch. > > --- > > libavcodec/x86/vp9lpf.asm | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/libav

Re: [FFmpeg-devel] [PATCH 07/15] vp8/x86: move variable assigned inside macro branch.

2014-12-27 Thread Clément Bœsch
On Sat, Dec 27, 2014 at 11:02:42AM -0500, Ronald S. Bultje wrote: > The value is not used outside the branch. > --- > libavcodec/x86/vp9lpf.asm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/x86/vp9lpf.asm b/libavcodec/x86/vp9lpf.asm > index e117798..15d4159 1

Re: [FFmpeg-devel] [PATCH 07/15] vp8/x86: move variable assigned inside macro branch.

2014-12-27 Thread Timothy Gu
On Saturday, December 27, 2014, Ronald S. Bultje wrote: > The value is not used outside the branch. > --- > libavcodec/x86/vp9lpf.asm | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Is this the right patch? Timothy ___ ffmpeg-devel mailing

[FFmpeg-devel] [PATCH 07/15] vp8/x86: move variable assigned inside macro branch.

2014-12-27 Thread Ronald S. Bultje
The value is not used outside the branch. --- libavcodec/x86/vp9lpf.asm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/x86/vp9lpf.asm b/libavcodec/x86/vp9lpf.asm index e117798..15d4159 100644 --- a/libavcodec/x86/vp9lpf.asm +++ b/libavcodec/x86/vp9lpf.asm @@ -469,8