Re: [FFmpeg-devel] [PATCH] avformat/movenc: use unspecified language by default

2019-05-23 Thread James Almer
On 5/23/2019 4:32 PM, Marton Balint wrote:
> 
> 
> On Mon, 20 May 2019, Carl Eugen Hoyos wrote:
> 
>> Am So., 19. Mai 2019 um 20:10 Uhr schrieb Marton Balint :
>>
>>> English was used before.
>>
>> A version bump is necessary.
>>
>> I would also like a Changelog entry, most people prefer an entry to
>> apichanges though.
> 
> I can do both.

As i said, no APIChanges. This doesn't introduce nor deprecate any
public symbol. It's a change of default behavior for a muxer, so micro
bump and Changelog entry is enough.

> Do people prefer to delay this until 4.2 is branched?
> 
> Thanks,
> Marton
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avformat/movenc: use unspecified language by default

2019-05-23 Thread Marton Balint



On Mon, 20 May 2019, Carl Eugen Hoyos wrote:


Am So., 19. Mai 2019 um 20:10 Uhr schrieb Marton Balint :


English was used before.


A version bump is necessary.

I would also like a Changelog entry, most people prefer an entry to
apichanges though.


I can do both. Do people prefer to delay this until 4.2 is branched?

Thanks,
Marton
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avformat/movenc: use unspecified language by default

2019-05-20 Thread James Almer
On 5/20/2019 6:58 AM, Carl Eugen Hoyos wrote:
> Am So., 19. Mai 2019 um 20:10 Uhr schrieb Marton Balint :
> 
>> English was used before.
> 
> A version bump is necessary.
> 
> I would also like a Changelog entry, most people prefer an entry to
> apichanges though.

No API is being changed or added, so this doesn't belong in APIChanges.

Also, i personally don't consider changing a default value for a
metadata field on a single muxer something worth mentioning in
Changelog, but if you think it's worth it then i won't oppose.

> 
> Carl Eugen
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> 
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
> 

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Re: [FFmpeg-devel] [PATCH] avformat/movenc: use unspecified language by default

2019-05-20 Thread Carl Eugen Hoyos
Am So., 19. Mai 2019 um 20:10 Uhr schrieb Marton Balint :

> English was used before.

A version bump is necessary.

I would also like a Changelog entry, most people prefer an entry to
apichanges though.

Carl Eugen
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH] avformat/movenc: use unspecified language by default

2019-05-19 Thread Marton Balint
English was used before.

Signed-off-by: Marton Balint 
---
 libavformat/movenc.c   | 2 +-
 tests/ref/acodec/alac  | 2 +-
 tests/ref/acodec/pcm-s16be | 2 +-
 tests/ref/acodec/pcm-s24be | 2 +-
 tests/ref/acodec/pcm-s32be | 2 +-
 tests/ref/acodec/pcm-s8| 2 +-
 tests/ref/fate/adtstoasc_ticket3715| 2 +-
 tests/ref/lavf/mov | 6 +++---
 tests/ref/lavf/mov_rtphint | 2 +-
 tests/ref/vsynth/vsynth1-avui  | 2 +-
 tests/ref/vsynth/vsynth1-dnxhd-1080i   | 2 +-
 tests/ref/vsynth/vsynth1-dnxhd-1080i-10bit | 2 +-
 tests/ref/vsynth/vsynth1-dnxhd-1080i-colr  | 2 +-
 tests/ref/vsynth/vsynth1-dnxhd-hr-hq-mov   | 2 +-
 tests/ref/vsynth/vsynth1-dnxhd-hr-lb-mov   | 2 +-
 tests/ref/vsynth/vsynth1-dnxhd-hr-sq-mov   | 2 +-
 tests/ref/vsynth/vsynth1-mov-bgr24 | 2 +-
 tests/ref/vsynth/vsynth1-mov-bpp15 | 2 +-
 tests/ref/vsynth/vsynth1-mov-bpp16 | 2 +-
 tests/ref/vsynth/vsynth1-prores| 2 +-
 tests/ref/vsynth/vsynth1-prores_444| 2 +-
 tests/ref/vsynth/vsynth1-prores_444_int| 2 +-
 tests/ref/vsynth/vsynth1-prores_int| 2 +-
 tests/ref/vsynth/vsynth1-prores_ks | 2 +-
 tests/ref/vsynth/vsynth1-qtrle | 2 +-
 tests/ref/vsynth/vsynth1-qtrlegray | 2 +-
 tests/ref/vsynth/vsynth1-svq1  | 2 +-
 tests/ref/vsynth/vsynth1-vc2-420p  | 2 +-
 tests/ref/vsynth/vsynth1-vc2-420p10| 2 +-
 tests/ref/vsynth/vsynth1-vc2-420p12| 2 +-
 tests/ref/vsynth/vsynth1-vc2-422p  | 2 +-
 tests/ref/vsynth/vsynth1-vc2-422p10| 2 +-
 tests/ref/vsynth/vsynth1-vc2-422p12| 2 +-
 tests/ref/vsynth/vsynth1-vc2-444p  | 2 +-
 tests/ref/vsynth/vsynth1-vc2-444p10| 2 +-
 tests/ref/vsynth/vsynth1-vc2-444p12| 2 +-
 tests/ref/vsynth/vsynth1-vc2-t5_3  | 2 +-
 tests/ref/vsynth/vsynth1-vc2-thaar | 2 +-
 tests/ref/vsynth/vsynth2-avui  | 2 +-
 tests/ref/vsynth/vsynth2-dnxhd-1080i   | 2 +-
 tests/ref/vsynth/vsynth2-dnxhd-1080i-10bit | 2 +-
 tests/ref/vsynth/vsynth2-dnxhd-1080i-colr  | 2 +-
 tests/ref/vsynth/vsynth2-dnxhd-hr-hq-mov   | 2 +-
 tests/ref/vsynth/vsynth2-dnxhd-hr-lb-mov   | 2 +-
 tests/ref/vsynth/vsynth2-dnxhd-hr-sq-mov   | 2 +-
 tests/ref/vsynth/vsynth2-mov-bgr24 | 2 +-
 tests/ref/vsynth/vsynth2-mov-bpp15 | 2 +-
 tests/ref/vsynth/vsynth2-mov-bpp16 | 2 +-
 tests/ref/vsynth/vsynth2-prores| 2 +-
 tests/ref/vsynth/vsynth2-prores_444| 2 +-
 tests/ref/vsynth/vsynth2-prores_444_int| 2 +-
 tests/ref/vsynth/vsynth2-prores_int| 2 +-
 tests/ref/vsynth/vsynth2-prores_ks | 2 +-
 tests/ref/vsynth/vsynth2-qtrle | 2 +-
 tests/ref/vsynth/vsynth2-qtrlegray | 2 +-
 tests/ref/vsynth/vsynth2-svq1  | 2 +-
 tests/ref/vsynth/vsynth2-vc2-420p  | 2 +-
 tests/ref/vsynth/vsynth2-vc2-420p10| 2 +-
 tests/ref/vsynth/vsynth2-vc2-420p12| 2 +-
 tests/ref/vsynth/vsynth2-vc2-422p  | 2 +-
 tests/ref/vsynth/vsynth2-vc2-422p10| 2 +-
 tests/ref/vsynth/vsynth2-vc2-422p12| 2 +-
 tests/ref/vsynth/vsynth2-vc2-444p  | 2 +-
 tests/ref/vsynth/vsynth2-vc2-444p10| 2 +-
 tests/ref/vsynth/vsynth2-vc2-444p12| 2 +-
 tests/ref/vsynth/vsynth2-vc2-t5_3  | 2 +-
 tests/ref/vsynth/vsynth2-vc2-thaar | 2 +-
 tests/ref/vsynth/vsynth3-dnxhd-1080i-10bit | 2 +-
 tests/ref/vsynth/vsynth3-dnxhd-1080i-colr  | 2 +-
 tests/ref/vsynth/vsynth3-dnxhd-hr-hq-mov   | 2 +-
 tests/ref/vsynth/vsynth3-dnxhd-hr-lb-mov   | 2 +-
 tests/ref/vsynth/vsynth3-dnxhd-hr-sq-mov   | 2 +-
 tests/ref/vsynth/vsynth3-mov-bgr24 | 2 +-
 tests/ref/vsynth/vsynth3-mov-bpp15 | 2 +-
 tests/ref/vsynth/vsynth3-mov-bpp16 | 2 +-
 tests/ref/vsynth/vsynth3-prores| 2 +-
 tests/ref/vsynth/vsynth3-prores_444| 2 +-
 tests/ref/vsynth/vsynth3-prores_444_int| 2 +-
 tests/ref/vsynth/vsynth3-prores_int| 2 +-
 tests/ref/vsynth/vsynth3-prores_ks | 2 +-
 tests/ref/vsynth/vsynth3-qtrle | 2 +-
 tests/ref/vsynth/vsynth3-svq1  | 2 +-
 tests/ref/vsynth/vsynth_lena-avui  | 2 +-
 tests/ref/vsynth/vsynth_lena-dnxhd-1080i   | 2 +-
 tests/ref/vsynth/vsynth_lena-dnxhd-1080i-10bit | 2 +-
 tests/ref/vsynth/vsynth_lena-dnxhd-1080i-colr  | 2 +-
 tests/ref/vsynth/vsynth_lena-dnxhd-hr-hq-mov   | 2 +-
 tests/ref/vsynth/vsynth_lena-dnxhd-hr-lb-mov   | 2 +-
 tests/ref/vsynth/vsynth_lena-dnxhd-hr-sq-mov   | 2 +-
 tests/ref/vsynth/vsynth_lena-mov-bgr24