On Sun, Mar 29, 2015 at 10:07:21PM -0600, Rodger Combs wrote:
> This needs a fair bit of testing and review before merge.
> 
> Re: mini:
      ^^^^
If you wish to shorten my name please use mn, its fewer keys, so you
safe some time


> > if the header does get updated at the end this would mismatch if only a
> > subset of segments get concatenated
> 
> This is one reason why I have the `seekback` option disabled by default
> (the other being that it avoids potential interprocess races)

ok, i tested a bit and it seems:
./ffmpeg -i randomfile.mpg -f ssegment -t 4 -segment_time 1.5  
-individual_header_trailer 0 file.%4d.ts
results in a missing file.0000.ts segments starting from 0001 instead

[...]


>      /* copy modified name in list entry */
>      size = strlen(av_basename(oc->filename)) + 1;
> +
>      if (seg->entry_prefix)
>          size += strlen(seg->entry_prefix);
>  
> @@ -226,19 +355,23 @@ static int segment_start(AVFormatContext *s, int 
> write_header)
>      }
>  
>      seg->segment_idx++;
> -    if ((seg->segment_idx_wrap) && (seg->segment_idx%seg->segment_idx_wrap 
> == 0))
> +    if ((seg->segment_idx_wrap) && (seg->segment_idx % seg->segment_idx_wrap 
> == 0))

unrelated

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

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to