[FFmpeg-devel] [PATCH v2] avcodec/dnxhdenc: remove the unneed *thread[MAX_THREADS]

[Limin Wang]

On Sun, Sep 22, 2019 at 01:04:01PM -0300, James Almer wrote:
> On 9/22/2019 1:01 PM, Limin Wang wrote:
> > On Sun, Sep 22, 2019 at 12:45:48PM -0300, James Almer wrote:
> >> On 9/22/2019 12:43 PM, lance.lmwang at gmail.com wrote:
> >>> From: Limin Wang <lance.lmwang at gmail.com>
> >>>
> >>> Signed-off-by: Limin Wang <lance.lmwang at gmail.com>
> >>> ---
> >>>  libavcodec/dnxhdenc.c | 47 ++++++++++-------------------------------------
> >>>  libavcodec/dnxhdenc.h |  2 --
> >>>  2 files changed, 10 insertions(+), 39 deletions(-)
> >>
> >> Err, what? I think you misunderstood what i called unnecessary in my
> >> last email...
> > No, I'm not familar with dnxhdenc too much, so I have pass "make fate" to testing
> > the function(with THREAD=0 and THREAD=2)
> > 
> > For the performance, by my quick testing, the results are same as before.
> > 
> > I hope it's my misunderstanding, but the results are OK.
> 
> Try with THREAD_TYPE=slice, otherwise you'll be testing frame threading.

OK, got it. I believe I haven't tested the code without it.

> 
> > 
> > 
> > Thanks,
> > Limin
> > 
> > 
> >> _______________________________________________
> >> ffmpeg-devel mailing list
> >> ffmpeg-devel at ffmpeg.org
> >> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >>
> >> To unsubscribe, visit link above, or email
> >> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> > 
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> > 
> 
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".