Quoting Diego Biurrun (2017-01-11 10:43:56) > On Wed, Jan 11, 2017 at 10:16:55AM +0100, Anton Khirnov wrote: > > Quoting wm4 (2017-01-11 08:08:49) > > > On Tue, 10 Jan 2017 15:42:33 +0100 > > > Anton Khirnov <[email protected]> wrote: > > > > --- a/doc/multithreading.txt > > > > +++ b/doc/multithreading.txt > > > > @@ -58,9 +58,10 @@ Add CODEC_CAP_FRAME_THREADS to the codec > > > > capabilities. There will be very little > > > > > > > > If there are inter-frame dependencies, so the codec calls > > > > -ff_thread_report/await_progress(), set > > > > AVCodecInternal.allocate_progress. The > > > > +ff_thread_report/await_progress(), set FF_CODEC_CAP_ALLOCATE_PROGRESS > > > > in > > > > +AVCodec.caps_internal and use ff_thread_get_buffer() to allocate > > > > frames. The > > > > > > The sentence doesn't look like proper English (both before and after > > > your change) - maybe that could be fixed too? > > > > Why? It looks rather normal to me. > > Umm, no :) > > Here's the sentence with third person singular 's' and s/so/then/: > > If there are inter-frame dependencies, then the codec calls > ff_thread_report/await_progress(), sets FF_CODEC_CAP_ALLOCATE_PROGRESS in > AVCodec.caps_internal, and uses ff_thread_get_buffer() to allocate frames. > > Probably s/calls/has to call/ would be even better.
I still maintain that the original form was correct (though perhaps somewhat awkward). In any case, it's unrelated to the patch. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
