[FFmpeg-devel] [PATCH]Two patches from github

wm4 nfxjfg at googlemail.com
Tue May 16 14:24:27 EEST 2017


On Tue, 16 May 2017 12:40:03 +0200
Carl Eugen Hoyos <ceffmpeg at gmail.com> wrote:

> 2017-05-15 12:53 GMT+02:00 wm4 <nfxjfg at googlemail.com>:
> > On Mon, 15 May 2017 12:40:23 +0200
> > Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> >  
> >> From 42766f345dbf398716c6fd9072f072f5fa91c940 Mon Sep 17 00:00:00 2001
> >> From: Steve Kondik <steve at cyngn.com>
> >> Date: Tue, 16 Dec 2014 01:37:57 -0800
> >> Subject: [PATCH 2/2] avutil: Use _SC_NPROCESSORS_CONF
> >>
> >>  * On most Android devices, CPUs can appear and disappear due to hotplug
> >>    or CPU cluster management. Use the total number of CPUs instead so
> >>    that multithreaded decoding is properly optimized.  
> >
> > This explanation isn't really sufficient.  
> 
> Please suggest a better one assuming you are objecting pushing as-is.

Yes, I'm objecting. I don't have to suggest a better one - you're the
one who wants to push this.

> > Also not why you would want
> > this, or why you would want this only on Android.  
> 
> The explanation sounded to me as if this would be Android-only.

The explanation is completely insufficient.


More information about the ffmpeg-devel mailing list