On Thu, Sep 8, 2011 at 14:49, Diego Biurrun <di...@biurrun.de> wrote:

> On Thu, Sep 08, 2011 at 12:58:26PM +0300, avia...@gmail.com wrote:
> >
> > --- a/libavformat/mpegts.c
> > +++ b/libavformat/mpegts.c
> > @@ -1413,10 +1413,10 @@ static int handle_packets(MpegTSContext *ts, int
> nb_packets)
> >          for (i = 0; i < NB_PID_MAX; i++) {
> >              if (ts->pids[i]) {
> >                  if (ts->pids[i]->type == MPEGTS_PES) {
> > -                PESContext *pes = ts->pids[i]->u.pes_filter.opaque;
> > -                av_freep(&pes->buffer);
> > -                pes->data_index = 0;
> > -                pes->state = MPEGTS_SKIP; /* skip until pes header */
> > +                   PESContext *pes = ts->pids[i]->u.pes_filter.opaque;
> > +                   av_freep(&pes->buffer);
> > +                   pes->data_index = 0;
> > +                   pes->state = MPEGTS_SKIP; /* skip until pes header */
>
> I'd suggest squashing this patch into 1/3.
>
>
I thought it was standard procedure to do whitespace changes like reindent
in separate patches?

-- 
Aviad Rozenhek
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to