[FFmpeg-devel] [PATCH] Add default stream flag

Evgeniy Stepanov eugeni.stepanov
Sun Mar 2 02:20:55 CET 2008


On Friday 29 February 2008 16:58:36 Michael Niedermayer wrote:
> On Fri, Feb 29, 2008 at 05:46:59AM +0300, Evgeniy Stepanov wrote:
> > On Friday 29 February 2008 04:51:04 Michael Niedermayer wrote:
> > [...]
> >
> > > > +static void set_disposition_bits(AVFormatContext* avf, char* value,
> > > > unsigned int stream_id_plus1){ +    int flag = 0, i;
> > > > +    for (i=0; ff_nut_dispositions[i].flag != 0; ++i) {
> > >
> > > the != 0 is superflous
> > >
> > > > +        if (!strncmp(ff_nut_dispositions[i].str, value,
> > > > +                     strlen(ff_nut_dispositions[i].str))) {
> > >
> > > why strncmp() and not strcmp() ?
> >
> > Mindless copypaste from matroska* )
>
> looks ok (maybe with a minor version bump for the disposition addition to
> AVStream)
>
> [...]

Could it be committed then ? Attached is the same patch with lavf minor 
version increased.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: stream-disposition.patch
Type: text/x-diff
Size: 7972 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080302/05cf7f9d/attachment.patch>



More information about the ffmpeg-devel mailing list