On 3/2/2021 7:38 AM, Andreas Rheinhardt wrote:
The current size is not enough, see e.g.
http://ftp.itec.aau.at/datasets/DASHDataset2014/TearsOfSteel/1sec/TearsOfSteel_1s_simple_2014_05_09.mpd

Signed-off-by: Andreas Rheinhardt <[email protected]>
---
I don't know if there is a fixed size limit or not (probably not),
but 32 is able to accomodate everything I found.

There is no defined limit, so maybe it should be dynamic? One could easily be even more verbose than what the above manifest did, and still be compliant.

And FWIW, what is however mentioned is that white spaces aren't allowed, so that manifest above is non-compliant.


  libavformat/dashdec.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/dashdec.c b/libavformat/dashdec.c
index 3a12aefa68..6b43fd6278 100644
--- a/libavformat/dashdec.c
+++ b/libavformat/dashdec.c
@@ -82,7 +82,7 @@ struct representation {
      AVFormatContext *ctx;
      int stream_index;
- char id[20];
+    char id[32];
      char *lang;
      int bandwidth;
      AVRational framerate;


_______________________________________________
ffmpeg-devel mailing list
[email protected]
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
[email protected] with subject "unsubscribe".

Reply via email to