On 28 Sep 2014, at 22:37, Vittorio Giovara <vittorio.giov...@gmail.com> wrote:

> On Sun, Sep 28, 2014 at 2:30 AM, Luca Barbato <lu_z...@gentoo.org> wrote:
>> On 26/09/14 18:22, Vittorio Giovara wrote:
>>> 
>>> ---
>> 
>> What it looks like
>> 
>>    Stream #0.0(eng): Video: hevc (Main)
>>      yuv420p, tv, 1920x1072, [PAR 1:1 DAR 120:67]
>>      23.98 fps, 1k tbn, 23.98 tbc (default)
>>    Stream #0.1(un): Audio: aac, 44100 Hz
>>      5.1, fltp (default)
>>    Stream #0.2(eng): Subtitle: [0][0][0][0] / 0x0000 (default)
>>    Stream #0.3(eng): Subtitle: [0][0][0][0] / 0x0000
> 
> I updated the original patch to put samplerate on a new line, like you
> requested.
> 
>> I'd change it so it looks
>> 
>>    Stream #0.0(eng): Video: hevc (Main) [default]
>>        yuv420p, tv, 1920x1072, [PAR 1:1 DAR 120:67]
>>        23.98 fps, 1k tbn, 23.98 tbc
>>    Stream #0.1(un): Audio: aac [default]
>>        44100 Hz, 5.1, fltp
>>    Stream #0.2(eng): Subtitle: [0][0][0][0] / 0x0000 [default]
>>    Stream #0.3(eng): Subtitle: [0][0][0][0] / 0x0000
> 
> Moving disposition tag should be on a different patch and i'd rather
> leave the first line only for codec and profile information. Actually
> I have never seen a sample with a disposition different than
> "default", is it worthwhile printing it at all?

Yes. There usually is only one default track of each kind (video, audio, 
subtitle) and knowing which track is (or, if there are multiple default tracks 
of the same type, which is OK in some containers) is worthwhile IMO.

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

Reply via email to