[Gnash-commit] gnash/libmedia FLVParser.cpp

[Sandro Santilli]

CVSROOT:        /sources/gnash
Module name:    gnash
Changes by:     Sandro Santilli <strk>  08/06/04 17:13:36

Modified files:
        libmedia       : FLVParser.cpp 

Log message:
        add note about probably bogus assumption in 
next{Video,Audio}FrameTimestamp impl

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/gnash/libmedia/FLVParser.cpp?cvsroot=gnash&r1=1.11&r2=1.12

Patches:
Index: FLVParser.cpp
===================================================================
RCS file: /sources/gnash/gnash/libmedia/FLVParser.cpp,v
retrieving revision 1.11
retrieving revision 1.12
diff -u -b -r1.11 -r1.12
--- FLVParser.cpp       4 Jun 2008 16:49:23 -0000       1.11
+++ FLVParser.cpp       4 Jun 2008 17:13:35 -0000       1.12
@@ -198,7 +198,14 @@
 FLVParser::nextAudioFrameTimestamp(boost::uint64_t& ts)
 {
        // If there are no audio in this FLV return NULL
-       if (!_audio && _lastParsedPosition > 0) return false;
+       // 
+       // TODO: FIXME: the condition assumes that if _lastParsedPosition > 0
+       //       we had a chance to figure if video was present !
+       //
+       if (!_audio && _lastParsedPosition > 0)
+       {
+               return false;
+       }
 
        // Make sure that there are parsed enough frames to return the need 
frame
        while(_audioFrames.size() <= _nextAudioFrame && !_parsingComplete) {
@@ -239,9 +246,12 @@
 FLVParser::nextVideoFrameTimestamp(boost::uint64_t& ts)
 {
        // If there are no video in this FLV return NULL
+       // 
+       // TODO: FIXME: the condition assumes that if _lastParsedPosition > 0
+       //       we had a chance to figure if video was present !
+       //
        if (!_video && _lastParsedPosition > 0)
        {
-               //gnash::log_debug("no video, or lastParserPosition > 0");
                return false;
        }