On Tue, 15 Oct 2013 09:50:19 +0200, Diego Biurrun <[email protected]> wrote: > On Sat, Oct 12, 2013 at 11:55:44AM +0200, Anton Khirnov wrote: > > --- /dev/null > > +++ b/libavformat/hevcdec.c > > @@ -0,0 +1,64 @@ > > + > > +#include "avformat.h" > > +#include "rawdec.h" > > + > > +#include "libavcodec/hevc.h" > > This is the reverse order of what we agreed on recently I think. > > > + switch (type) { > > + case NAL_VPS: vps++; break; > > + case NAL_SPS: sps++; break; > > + case NAL_PPS: pps++; break; > > + case NAL_BLA_N_LP: > > + case NAL_BLA_W_LP: > > + case NAL_BLA_W_RADL: > > + case NAL_CRA_NUT: > > + case NAL_IDR_N_LP: > > + case NAL_IDR_W_RADL: irap++; break; > > Break the lines please. >
I do not think breaking the lines will make it more readable. -- Anton Khirnov _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
