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

[Michael Niedermayer]

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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20161129/4a935cf4/attachment.sig>