[FFmpeg-devel] [PATCH 2/3] hevc: Support extradata changes

Vittorio Giovara vittorio.giovara at gmail.com
Tue Nov 29 17:53:15 EET 2016


On Tue, Nov 29, 2016 at 7:20 AM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> On Mon, Nov 28, 2016 at 10:03:37PM -0500, Vittorio Giovara wrote:
>> On Mon, Nov 28, 2016 at 9:06 PM, Michael Niedermayer
>> <michael at niedermayer.cc> wrote:
>> > On Tue, Nov 08, 2016 at 05:03:27PM -0500, Vittorio Giovara wrote:
>> >> Signed-off-by: Vittorio Giovara <vittorio.giovara at gmail.com>
>> >> ---
>> >> Applied review.
>> >> Please CC.
>> >> Vittorio
>> >>
>> >>  libavcodec/hevc.c | 10 ++++++++++
>> >>  libavformat/mov.c |  4 ----
>> >
>> > please split this in 2 patches, the libavcodec one probably should
>> > also have its version bumped as apps might want to depend on
>> > a libavcodec with that feature
>>
>> ok for the version bumb, why splitting it 2 patches though?
>
> making changes to 2 libs at the same time can mask bugs because
> you cannot checkout and test the intermediate but in distributions
> users can end up with one lib updated and the other not (within what
> the dependancies and versions allow)
> So i always suggest spliting non cosmetic changes into a patch per lib
> unless i miss/forget

Well yes, but the changes in lavf only drop a log line, which is
useless because of this patch, so imo it makes sense to keep them in
the same diff. I'll split it if you insist though.
Thanks
-- 
Vittorio


More information about the ffmpeg-devel mailing list