[FFmpeg-devel] [PATCH] avformat/matroskaenc: add matroska subtitle muxer (.mks)

2020-04-03 Thread Jan Chren (rindeal)
RnJvbSBlYWE5ZjI2ODU0MzY3MjVlMTllYjk0ZTdmMmQwMTc0YmE1NmI3NDI5IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBKYW4gQ2hyZW4gKHJpbmRlYWwpIDxkZXYucmluZGVhbEBnbWFp bC5jb20+CkRhdGU6IFRodSwgMSBBcHIgMjAyMCAwMDowMDowMCArMDAwMApTdWJqZWN0OiBbUEFU Q0hdIGF2Zm9ybWF0L21hdHJvc2thZW5jOiBhZGQgbWF0cm9za2Egc3VidGl0bGUgbXV4

[FFmpeg-devel] [PATCH] avformat/matroskaenc: remove MAX_TRACKS limit

2020-04-03 Thread Jan Chren (rindeal)
RnJvbSA0M2U0Yjk2OTVhNDM0MjExMmIxMTE3ZGI2NTI2YmFmYTFmOWRhYjNlIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBKYW4gQ2hyZW4gKHJpbmRlYWwpIDxkZXYucmluZGVhbEBnbWFp bC5jb20+CkRhdGU6IFRodSwgMSBBcHIgMjAyMCAwMDowMDowMCArMDAwMApTdWJqZWN0OiBbUEFU Q0hdIGF2Zm9ybWF0L21hdHJvc2thZW5jOiByZW1vdmUgTUFYX1RSQUNLUyBsaW1pdAoK

[FFmpeg-devel] [PATCH] avformat/matroskaenc: remove MAX_TRACKS limit

2020-04-04 Thread Jan Chren (rindeal)
From 43e4b9695a4342112b1117db6526bafa1f9dab3e Mon Sep 17 00:00:00 2001 From: Jan Chren (rindeal) Date: Thu, 1 Apr 2020 00:00:00 + Subject: [PATCH] avformat/matroskaenc: remove MAX_TRACKS limit It was introduced in 7be0f48a32155ef9f471ffc5a1b41d662ea337f1 to set size of an array struct field

[FFmpeg-devel] [PATCH] avformat/matroskaenc: add matroska subtitle muxer (.mks)

2020-04-04 Thread Jan Chren (rindeal)
From eaa9f2685436725e19eb94e7f2d0174ba56b7429 Mon Sep 17 00:00:00 2001 From: Jan Chren (rindeal) Date: Thu, 1 Apr 2020 00:00:00 + Subject: [PATCH] avformat/matroskaenc: add matroska subtitle muxer (.mks) Signed-off-by: Jan Chren (rindeal) --- libavformat/matroskaenc.c | 28

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: add matroska subtitle muxer (.mks)

2020-04-04 Thread Jan Chren (rindeal)
On Sat, 4 Apr 2020 at 15:56, Paul B Mahol wrote: > > Looks like you sent exactly same patch as before. > This is actually the third time I sent it, but first time it went through to Patchwork, which is what counts. > This patch does unfortunately nothing as some changes from other files > are mi

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: add matroska subtitle muxer (.mks)

2020-04-04 Thread Jan Chren (rindeal)
On Sat, 4 Apr 2020 at 16:32, Andreas Rheinhardt wrote: > > You should read my answers to your earlier mails. Any earlier mails have not reached either my mailbox or Patchwork. But hopefully I have found them in the public mail archive now. Terrible workflow got even worse... > > - Andreas __

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: add matroska subtitle muxer (.mks)

2020-04-04 Thread Jan Chren (rindeal)
On Sat, 4 Apr 2020 at 16:32, Andreas Rheinhardt wrote: > What is the aim of your patch? If it is simply making sure that using > the mks extension autoselects the Matroska muxer, That's correct. > then adding "mks" to > the list of extensions of the ordinary Matroska muxer would be enough. > (No

Re: [FFmpeg-devel] [PATCH] avformat/matroskaenc: remove MAX_TRACKS limit

2020-04-04 Thread Jan Chren (rindeal)
attachments (in this case there may > be gaps in the used TrackNumbers). I have included a new patch, which writes the track_number using put_ebml_num(), which should work perhaps? From 477d1ee08115e09f2f61f75dd70ce7bc882d2c34 Mon Sep 17 00:00:00 2001 From: Jan Chren (rindeal) Date: Sat, 4 Apr 2020 0

Re: [FFmpeg-devel] [PATCH 12/20] avformat/matroskaenc: Improve Cues in case of no video

2020-04-05 Thread Jan Chren (rindeal)
On Sun, 5 Apr 2020 at 16:01, Andreas Rheinhardt wrote: > > The Matroska muxer currently only adds CuePoints in three cases: > a) For video keyframes. b) For the first audio frame in a new Cluster if > in DASH-mode. c) For subtitles. This means that ordinary Matroska audio > files won't have any Cu