[vlc-commits] dash: Don't require the Representation @id. Many stream won't provide it.

Hugo Beauzée-Luyssen git at videolan.org
Tue Jan 24 23:21:56 CET 2012


vlc/vlc-1.2 | branch: master | Hugo Beauzée-Luyssen <beauze.h at gmail.com> | Wed Dec 28 17:32:34 2011 +0100| [78822a604bee9d8dc0a08496d0a8755e414261fa] | committer: Jean-Baptiste Kempf

dash: Don't require the Representation @id. Many stream won't provide it.

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>
(cherry picked from commit 4abd446913081747a509e750b5c7e494c4d7a471)

Signed-off-by: Jean-Baptiste Kempf <jb at videolan.org>

> http://git.videolan.org/gitweb.cgi/vlc/vlc-1.2.git/?a=commit;h=78822a604bee9d8dc0a08496d0a8755e414261fa
---

 modules/stream_filter/dash/mpd/BasicCMParser.cpp |    7 ++-----
 1 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/modules/stream_filter/dash/mpd/BasicCMParser.cpp b/modules/stream_filter/dash/mpd/BasicCMParser.cpp
index c1db5ad..acafcb2 100644
--- a/modules/stream_filter/dash/mpd/BasicCMParser.cpp
+++ b/modules/stream_filter/dash/mpd/BasicCMParser.cpp
@@ -178,12 +178,9 @@ void    BasicCMParser::setRepresentations   (Node *root, Group *group)
 
         it = attributes.find( "id" );
         if ( it == attributes.end() )
-        {
             std::cerr << "Missing mandatory attribute for Representation: @id" << std::endl;
-            delete rep;
-            continue ;
-        }
-        rep->setId( it->second );
+        else
+            rep->setId( it->second );
 
         it = attributes.find( "bandwidth" );
         if ( it == attributes.end() )



More information about the vlc-commits mailing list