[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Gnash-commit] gnash/libmedia FLVParser.cpp
From: |
Sandro Santilli |
Subject: |
[Gnash-commit] gnash/libmedia FLVParser.cpp |
Date: |
Wed, 04 Jun 2008 17:13:36 +0000 |
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;
}
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Gnash-commit] gnash/libmedia FLVParser.cpp,
Sandro Santilli <=